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