llvm: assume ptrs have (at least) ABI alignment

AnyDSL / thorin

The Higher-Order Intermediate Representation

https://anydsl.github.io

GNU Lesser General Public License v3.0

150 stars 15 forks source link

llvm: assume ptrs have (at least) ABI alignment #99

Closed simoll closed 5 years ago

simoll commented 5 years ago

This patch sets the alignment of loads and stores in generated LLVM IR according to the target ABI.

I don't know whether ABI alignment is a valid assumption for Thorin codegen (thus this PR). However, this improves assembly for SX-Aurora (loads and stores are otherwise legalized down into byte loads and stores...).

leissa commented 5 years ago

Fine for me.

richardmembarth commented 5 years ago

This is also fine for me.