[llvm] [DAGCombiner] Don't drop atomic property of original load. (PR #75626)

Jonas Paulsson via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 12:32:34 PST 2023


JonPsson1 wrote:

It seems this patch actually shouldn't go in since it is probably not intentional to have DAGCombiner work on LOADs that are actually atomic. In other words, the MMO atomic state would generally be ignored with an ISD::LOAD, so if anything the SystemZ::lowerATOMIC_LOAD() should be improved. 

Ok to close this?


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


More information about the llvm-commits mailing list