[llvm-commits] [llvm] r148658 - in /llvm/trunk: lib/Target/ARM/ARM.td lib/Target/ARM/ARMAsmPrinter.cpp lib/Target/ARM/ARMInstrInfo.td lib/Target/ARM/ARMInstrNEON.td lib/Target/ARM/ARMInstrVFP.td lib/Target/ARM/ARMSchedule.td lib/Target/ARM/ARMSub

Anton Korobeynikov asl at math.spbu.ru
Sun Jan 22 10:15:00 PST 2012

Hi Bruno

On Sun, Jan 22, 2012 at 21:34, Bruno Cardoso Lopes
<bruno.cardoso at gmail.com> wrote:
>> +def FeatureVFP4 : SubtargetFeature<"vfp4", "HasVFPv4", "true",
>> +                                   "Enable VFP4 instructions",
>> +                                   [FeatureVFP3]>;
> You probably want "[FeatureVFP4]" here, right?
Surely not! This is "implied features" list. So, VFP4 implies VFP3.

With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University

More information about the llvm-commits mailing list