[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