[PATCH] D34311: [InstCombine] Don't replace allocas with smaller globals
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 23 17:22:12 PDT 2017
efriedma added inline comments.
================
Comment at: lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp:180
+ return false;
+ return isDereferenceableAndAlignedPointer(V, 1, APInt(64, AllocaSize), DL);
+}
----------------
Doesn't the alignment here need to be the alignment of the alloca, rather than "1"?
https://reviews.llvm.org/D34311
More information about the llvm-commits
mailing list