[PATCH] D109548: Make llvm.launder.invariant.group writeonly

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 10 02:44:11 PDT 2021


Prazek added inline comments.


================
Comment at: llvm/include/llvm/IR/Intrinsics.td:1156
 // because it would cause CSE of two barriers with the same argument.
 // Inaccessiblememonly says that the barrier doesn't read the argument,
 // but it changes state not accessible to this module. This way
----------------
Could you expand the comment here with writeonly?
Optional: s/barrier/launder


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109548



More information about the llvm-commits mailing list