[RISCV][NFC] Remove outdated TODOs about extension versions in RISCVISAInfo
authorAlex Bradbury <asb@igalia.com>
Wed, 26 Apr 2023 08:56:14 +0000 (09:56 +0100)
committerAlex Bradbury <asb@igalia.com>
Wed, 26 Apr 2023 08:58:57 +0000 (09:58 +0100)
As was documented in D147183, we don't currently intend to support
multiple versions of specifications simultaneously. While external
circumstances might mean exceptions to this are needed, the TODOs about
reflecting version numbers in the feature strings no longer make sense.

llvm/lib/Support/RISCVISAInfo.cpp

index 40cb45c..42fc2d3 100644 (file)
@@ -687,8 +687,6 @@ RISCVISAInfo::parseArchString(StringRef Arch, bool EnableExperimentalExtension,
   Exts = Exts.drop_front(ConsumeLength);
   Exts.consume_front("_");
 
-  // TODO: Use version number when setting target features
-
   auto StdExtsItr = StdExts.begin();
   auto StdExtsEnd = StdExts.end();
   auto GoToNextExt = [](StringRef::iterator &I, unsigned ConsumeLength) {
@@ -736,7 +734,6 @@ RISCVISAInfo::parseArchString(StringRef Arch, bool EnableExperimentalExtension,
     }
 
     // The order is OK, then push it into features.
-    // TODO: Use version number when setting target features
     // Currently LLVM supports only "mafdcvh".
     if (!isSupportedExtension(StringRef(&C, 1))) {
       if (IgnoreUnknown) {