[PATCH] D103073: [LTT] Handle merged llvm.assume when dropping type tests
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 28 21:36:13 PST 2022
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
More information about the llvm-commits
mailing list