[PATCH] D26641: [InstCombine] change bitwise logic type to eliminate bitcasts

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 18:13:56 PST 2016


spatel added a comment.

In https://reviews.llvm.org/D26641#602101, @efriedma wrote:

> 3 would be nice... but I'm not sure you can actually implement it properly without global isel because the operands/users could be defined in different blocks.


Hmm..good point. Could add to the CGP pile instead? But I'll propose that we use #2 with a FIXME comment for this patch, so we can make partial progress (I've only seen vector code with this problem so far).


https://reviews.llvm.org/D26641





More information about the llvm-commits mailing list