Skip to content

Commit 5d78c9c

Browse files
committed
Comment change. NFC
That comment misleads the current discussions in mentioned bug. Leave the discussions to the bug. Also, adding a future change FIXME. llvm-svn: 238653
1 parent cb58910 commit 5d78c9c

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9168,14 +9168,13 @@ bool ARMAsmParser::parseDirectiveCPU(SMLoc L) {
91689168
StringRef CPU = getParser().parseStringToEndOfStatement().trim();
91699169
getTargetStreamer().emitTextAttribute(ARMBuildAttrs::CPU_name, CPU);
91709170

9171+
// FIXME: This is using table-gen data, but should be moved to
9172+
// ARMTargetParser once that is table-gen'd.
91719173
if (!STI.isCPUStringValid(CPU)) {
91729174
Error(L, "Unknown CPU name");
91739175
return false;
91749176
}
91759177

9176-
// FIXME: This switches the CPU features globally, therefore it might
9177-
// happen that code you would not expect to assemble will. For details
9178-
// see: http://llvm.org/bugs/show_bug.cgi?id=20757
91799178
STI.InitMCProcessorInfo(CPU, "");
91809179
STI.InitCPUSchedModel(CPU);
91819180
setAvailableFeatures(ComputeAvailableFeatures(STI.getFeatureBits()));

0 commit comments

Comments
 (0)