[PATCH] D36840: [DAG] convert vector select-of-constants to logic/math

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 16:38:25 PDT 2017


spatel added a comment.

In https://reviews.llvm.org/D36840#844861, @spatel wrote:

> Patch updated:
>  Remove the general case fold which actually isn't a win if vselect is fast. We only convert the vselect when there's a clear optimization now.


That means we shouldn't need a TLI hook now, but I'd like to make that a follow-up because I'm not sure how other targets will change.


https://reviews.llvm.org/D36840





More information about the llvm-commits mailing list