[PATCH] D97848: [CodeGenPrepare] Eliminate llvm.expect before removing empty blocks

[Jann Horn via Phabricator via llvm-commits]

thejh marked an inline comment as done.
thejh added inline comments.


================
Comment at: llvm/test/Transforms/CodeGenPrepare/remove-assume-block.ll:11
+
+define void @a(i64 %addr, i1 %assumption) {
+  %cmp1 = icmp eq i64 %addr, 0
----------------
fhahn wrote:
> can you also test cases with blocks that contain multiple assumes, as well as tests where assume calls and other instructions are mixed. Also cases where the assumption is a value defined in the function.
Done - I think the test I've added covers all your requests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97848