[PATCH] D14588: [X86][SSE] Transform truncation from v8i32/v16i32 to v8i8/v16i8 into bitand and X86ISD::PACKUS operations during DAG combine.

Cong Hou via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 28 22:14:25 PST 2015


congh added a comment.

In http://reviews.llvm.org/D14588#298059, @RKSimon wrote:

> Please can you rebase this - your PAVG patches will have affected PerformTRUNCATECombine etc.


OK. I just found my implementation of using PACKSS is incorrect in this patch. I need to use shift-left then shift-right to set the bits of the higher part for negative values before using PACKSS. I will fix it soon.


http://reviews.llvm.org/D14588





More information about the llvm-commits mailing list