[PATCH] D97495: [SimplifyCFG] avoid illegal phi with both poison and undef
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 26 04:58:00 PST 2021
spatel marked an inline comment as done.
spatel added inline comments.
================
Comment at: llvm/test/Transforms/SimplifyCFG/poison-merge.ll:14-17
+; CHECK-NEXT: [[K2:%.*]] = phi i64 [ [[K3:%.*]], [[G]] ], [ undef, [[ENTRY:%.*]] ]
+; CHECK-NEXT: br label [[G]]
+; CHECK: g:
+; CHECK-NEXT: [[K3]] = phi i64 [ [[K2]], [[LOOP]] ], [ poison, [[ENTRY]] ]
----------------
lebedev.ri wrote:
> It looks like we didn't merge them after all?
That's correct - let me add some tests of the other value combos to make it clearer what this patch does.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97495/new/
https://reviews.llvm.org/D97495
More information about the llvm-commits
mailing list