[llvm] [VPlan] Fix alias logic in canHoistOrSinkWithNoAliasCheck (PR #179504)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 11 01:42:27 PST 2026


================
@@ -223,7 +223,7 @@ canHoistOrSinkWithNoAliasCheck(const MemoryLocation &MemLoc,
 
       // For reads, check if they don't alias in the reverse direction and
       // skip if so.
-      if (CheckReads && R.mayReadFromMemory() &&
+      if (!(CheckReads && R.mayReadFromMemory()) ||
----------------
artagnon wrote:

I think this logic is broken: I have an amendment to the logic locally, but the (bad?) change is still present, and I'm confused?

https://github.com/llvm/llvm-project/pull/179504


More information about the llvm-commits mailing list