arsenm wrote: > The duplicating is only due to atomicity of the load?
The duplicating is the apparent effect because the non-atomic load can be deleted. There's no plus to keeping it around > if (atomic) > EraseFromParent(); > ``` This just adds extra work for later code to delete. The zextload equivalent combine already directly deletes the load https://github.com/llvm/llvm-project/pull/111730 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
