[PATCH] D81230: Correctly report modified status for LoopIdiomRecognize

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 5 14:36:05 PDT 2020


nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:1118
     const MaybeAlign LoadAlign = LI->getAlign();
-    if (StoreAlign == None || LoadAlign == None)
+    if (StoreAlign == None || LoadAlign == None) {
+      Expander.clear();
----------------
asbirlea wrote:
> Similarly, merge conditions and comment appropriately.
This is actually dead code, I've dropped it in bff94a8e2bb93267a561ca96287f570af499b090.


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

https://reviews.llvm.org/D81230





More information about the llvm-commits mailing list