[PATCH] D38316: [InstCombine] replace bitcast to scalar + insertelement with widening shuffle + vector bitcast

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 27 11:20:57 PDT 2017


efriedma added a comment.

> Right - I thought about that case, but I couldn't justify it if we need another IR instruction.

So how do you plan to fix x86 lowering for this case?  If we need a DAGCombine for this anyway, not sure there's much point to handling it in instcombine.


https://reviews.llvm.org/D38316





More information about the llvm-commits mailing list