[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