[PATCH] D10905: move DAGCombiner's allowableAlignment() helper function into the TLI
Quentin Colombet
qcolombet at apple.com
Tue Jul 28 14:29:58 PDT 2015
qcolombet added inline comments.
================
Comment at: lib/CodeGen/TargetLoweringBase.cpp:1539
@@ +1538,3 @@
+ // Check if the specified alignment is sufficient based on the data layout.
+ Type *Ty = VT.getTypeForEVT(Context);
+ if (Alignment >= DL.getABITypeAlignment(Ty)) {
----------------
I’d say that’s fine for the default behavior, but we could provide a way to change that at the target discretion by making the hook virtual.
http://reviews.llvm.org/D10905
More information about the llvm-commits
mailing list