[PATCH] D152358: [CGP] Remove operand of llvm.assume more aggressively.

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 15 07:24:44 PDT 2023


anna added a comment.

LGTM (maybe wait for other reviewers for couple of days) with added tests.



================
Comment at: llvm/test/Transforms/CodeGenPrepare/X86/delete-assume-dead-code.ll:48
   %iv.inc = add i32 %iv, 1
   %iv2.inc = add i32 %iv2, 1
   %cmp = icmp eq i32 %iv, %N
----------------
Pls add couple of tests:
- positive test: %iv2.inc is used in another cmp which feeds into another assume
- negative test: %iv2 is used in a cmp3 which is used as a sub condition in the branch (and i1 %cmp, %cmp3)


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

https://reviews.llvm.org/D152358



More information about the llvm-commits mailing list