[llvm] [LAA][NFC] Factor out MemoryDepChecker::isStoreLoadForwardingConflict (PR #212085)
Milin Bhade via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 26 22:50:59 PDT 2026
================
----------------
mbhade-amd wrote:
Asserting isn't safe here: VF = 2 * TypeByteSize; VF *= 2 overflows to 0 for huge types, so VectorStoreSize == 0 is reachable (SIGFPE on trunk). A helper-only guard just turns the crash into a hang, so the real fix is stopping the loop on overflow (VF && VF <= bound).
That makes it more than a refactor — would you prefer I keep this PR strictly NFC and fix the overflow separately, or drop [NFC] and harden it here (loop guard + defensive return false + a LAA test)?
https://github.com/llvm/llvm-project/pull/212085
More information about the llvm-commits
mailing list