[llvm-commits] [llvm] r48491 - in /llvm/trunk: lib/CodeGen/SelectionDAG/DAGCombiner.cpp test/CodeGen/X86/field-extract-use-trunc.ll

[Christopher Lamb]

DAGCombine runs both before and after legalize, so I don't think you  
can assume that MVT::i1 will be promoted. As I said elsewhere,  
checking if the sext operation is legal for the type didn't fix the  
problem.

On Mar 19, 2008, at 2:11 AM, Duncan Sands wrote:

> Hi Evan,
>
>> Well... Actually it should return true. MVT::i1 is promoted to  
>> MVT::i8
>> so on x86 it really should be free.
>
> some operations involving i1 are considered legal on x86, for example
> an extending load from i1.  Maybe this has something to do with it.
>
> Ciao,
>
> Duncan.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

--
Christopher Lamb



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20080319/6ad82e49/attachment.html>