[PATCH] D141277: [InstCombine] Don't optimize idempotent `atomicrmw <op>, 0` into `load atomic`

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 19 01:22:08 PST 2023


qcolombet added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/atomicrmw.ll:5
 ; Check that we can replace `atomicrmw <op> LHS, 0` with `load atomic LHS`.
 ; This is possible when:
+; Check that we don't replace `atomicrmw <op> LHS, 0` with `load atomic LHS`.
----------------
nikic wrote:
> I think the two comment lines above were supposed to be dropped.
Good catch, looks like I screwed up the merge conflict.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141277/new/

https://reviews.llvm.org/D141277



More information about the llvm-commits mailing list