[llvm] [GVN] MemorySSA for GVN: embed the memory state in symbolic expressions (PR #123218)

Antonio Frighetto via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 16 08:31:51 PST 2025


================
@@ -596,10 +617,37 @@ uint32_t GVNPass::ValueTable::lookupOrAddCall(CallInst *C) {
     return v;
   }
 
+  if (MSSA && AA->onlyReadsMemory(C)) {
----------------
antoniofrighetto wrote:

Think this (and the one below) should check isMemorySSAEnabled too as MSSA and MemDep may be both enabled.

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


More information about the llvm-commits mailing list