[PATCH] D100470: [LICM][NFC] Fix typo

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:561
 
-      // If the instruction is dead, we would try to sink it because it isn't
+      // If the instruction is dead, we wouldn't try to sink it because it isn't
       // used in the loop, instead, just delete it.
----------------
Marcythm wrote:
> I'm not sure if this is right, but I think it should be closer to what the author wants to express.
I don't think the new wording is right: What this is saying is that without this check, the instruction would be sunk, which is legal but pointless. Instead it is completely removed here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100470