[PATCH] D103073: [LTT] Handle merged llvm.assume when dropping type tests

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.
Herald added a subscriber: ormris.


================
Comment at: llvm/test/Transforms/LowerTypeTests/drop_type_test_phi.ll:38
+  %4 = phi i1 [ %3, %if.else ], [ %p, %if.then ]
+  call void @llvm.assume(i1 %4)
+; Still have the assume, but the type test target replaced with true.
----------------
I am a bit puzzled by the comment "If this type test is only used by llvm.assume instructions": this test has exactly one assume. Where did the merge happen?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103073