[PATCH] D69086: [IR] Fix mayReadFromMemory() for writeonly calls

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 10:39:34 PDT 2019


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with minor test change



================
Comment at: llvm/test/Transforms/EarlyCSE/writeonly.ll:8
+; CHECK-LABEL: @test(
+; CHECK-NOT: store i32 1, i32* @var
+  store i32 1, i32* @var
----------------
I would loosen this to just CHECK-NOT: store in case of future IR syntax changes


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

https://reviews.llvm.org/D69086





More information about the llvm-commits mailing list