[PATCH] D13953: Preserve load alignment and dereferenceable metadata during some transformations

Artur Pilipenko via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 10:44:15 PST 2015


apilipenko added inline comments.

================
Comment at: lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp:376
@@ -375,1 +375,3 @@
       break;
+    case LLVMContext::MD_align:
+    case LLVMContext::MD_dereferenceable:
----------------
hfinkel wrote:
> Move these up to combine with nonnull handling.
nonnull code also contains logic for the case when new type is an integer. These can be combined using fall through case, but I don't think that it helps readability.


Repository:
  rL LLVM

http://reviews.llvm.org/D13953





More information about the llvm-commits mailing list