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

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 05:28:04 PST 2023


jyknight wrote:

> I think this should still go ahead. I still think it's a good idea to get rid of ISD::ATOMIC_LOAD and migrate to using load with atomic operands. 

Not sure I agree with that.

> This is a code simplification anyway

Agreed, this patch is good to submit as a simplification and bugfix for the alignment, regardless of the issues around ATOMIC_LOAD.

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


More information about the llvm-commits mailing list