[llvm] [GVN] Add MemorySSA checks in tests 1/N (PR #130261)

Madhur Amilkanthwar via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 20 08:11:12 PDT 2025


madhur13490 wrote:

> > > This looks reasonable to me now. @antoniofrighetto Any concerns with testing MSSA? Also, do you think it would make sense to make `gvn<memoryssa>` work by having it imply `no-memdep`?
> > 
> > 
> > I second this. Either of them will be in use, so enabling one should imply the disablement of the other.
> 
> That makes sense. I think it would be ideal to do this after flipping the flag.

Yes, it is ideal but I think we need to ensure what is passing should continue to pass. When we flip the switch, we know what is failing. I will add memoryssa to only those tests which are currently passing. For the failing test cases, I will file Github issue with appropriate keywords. 

Does this make sense?


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


More information about the llvm-commits mailing list