[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