On Jun 29, 2007, at 12:47 PM, Dan Gohman wrote:

>>>
>>> I notice that several places use !MVT::isExtendedValueType(VT).
>>> Please add MVT::isSimpleValueType()
>
> After making a different change to TargetLowering.h following the  
> review feedback,
> there are fewer of these. Fewer uses of isExtendedVT are negated  
> than not now.
> I'd prefer to avoid having two different names to do essentially  
> the same thing.

Ok, nice work on 8 x float btw!

-Chris
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to