[llvm] [SelectionDAG] Preserve volatile undef stores. (PR #99918)

Matt Davis via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 7 04:33:28 PDT 2024


enferex wrote:

> Should this be backpeoted to 19.x? Since something as big as volatile could have huge effects on codegen?

@RSilicon , this "issue" of the DAGCombiner dropping volatile undef stores has been around for quite a while, 10+years.  I'm not sure we want to back port this, I'm happy to do so, I do see one [bug ](https://github.com/llvm/llvm-project/issues/27683).  However, given it's been around for a while,  I can also see where it makes sense to just take it on as a fix for 20.  

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


More information about the llvm-commits mailing list