[PATCH] D76113: [ExpandMemCmp] Correctly set alignment of generated loads

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 03:53:21 PDT 2020


gchatelet added inline comments.


================
Comment at: llvm/lib/CodeGen/ExpandMemCmp.cpp:276
   Value *RhsSource = CI->getArgOperand(1);
+  Align LhsAlign = max(LhsSource->getPointerAlignment(DL), Align(1));
+  Align RhsAlign = max(RhsSource->getPointerAlignment(DL), Align(1));
----------------
`LhsSource->getPointerAlignment(DL).valueOrOne()`
and below as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76113/new/

https://reviews.llvm.org/D76113





More information about the llvm-commits mailing list