khchen added a comment. In D126745#3691544 <https://reviews.llvm.org/D126745#3691544>, @craig.topper wrote:
> In D126745#3691528 <https://reviews.llvm.org/D126745#3691528>, @khchen wrote: > >> In D126745#3678776 <https://reviews.llvm.org/D126745#3678776>, @nlopes wrote: >> >>> While at it, could you switch those UndefValue with PoisonValue if >>> possible? Thank you! >> >> I'm not sure UndefValue will work because the backend would only check >> isUndef() to generate correct code... > > PoisonValue and UndefValue are both turned into undef in SelectionDAG today. thanks @craig.topper, I will switch UndefValue with PoisonValue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126745/new/ https://reviews.llvm.org/D126745 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
