[PATCH] D47686: [InstCombine] refine UB-handling in shuffle-binop transform

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 3 09:10:35 PDT 2018


spatel added a comment.

In https://reviews.llvm.org/D47686#1120186, @lebedev.ri wrote:

> This makes sense to me, but i'm not really familiar with `shufflevector`,
>  so i'm going to leave the actual LGTM to others.


No problem - thanks for the suggestions! @efriedma noticed the potential holes in my initial fix, so I'll certainly wait for him to comment.


https://reviews.llvm.org/D47686





More information about the llvm-commits mailing list