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

[Jonas Paulsson via llvm-commits]

JonPsson1 wrote:

> getAlign() combines the base alignment of the MMO with the MMO offset. This can cause the base alignment of the newly created MMO to become smaller than the original MMO. This can produce different results if the MMO is later split into multiple MMOs with different offsets.

I don't quite follow how this could go wrong as folding the bitcast doesn't change the memory access in any way. Shouldn't the later splitting also take into account the original offset?


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