[llvm] a587f42 - [TableGen] SubtargetEmitter must use std::nullopt (#69475)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 18 10:11:27 PDT 2023


Author: Kai Nacke
Date: 2023-10-18T13:11:23-04:00
New Revision: a587f42953930119d743da863458eaee45c6b70c

URL: https://github.com/llvm/llvm-project/commit/a587f42953930119d743da863458eaee45c6b70c
DIFF: https://github.com/llvm/llvm-project/commit/a587f42953930119d743da863458eaee45c6b70c.diff

LOG: [TableGen] SubtargetEmitter must use std::nullopt (#69475)

Use of llvm::Optional was migrated to std::optional. This included a
change in the constructor of ArrayRef.
However, there are still 2 places in the SubtargetEmitter which uses
llvm::None, causing a compile error when emitted.

Added: 
    

Modified: 
    llvm/utils/TableGen/SubtargetEmitter.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/utils/TableGen/SubtargetEmitter.cpp b/llvm/utils/TableGen/SubtargetEmitter.cpp
index 5e822078b947123..f7a7172d61fc618 100644
--- a/llvm/utils/TableGen/SubtargetEmitter.cpp
+++ b/llvm/utils/TableGen/SubtargetEmitter.cpp
@@ -1935,7 +1935,7 @@ void SubtargetEmitter::run(raw_ostream &OS) {
   if (NumProcs)
     OS << Target << "SubTypeKV, ";
   else
-    OS << "None, ";
+    OS << "std::nullopt, ";
   OS << '\n'; OS.indent(22);
   OS << Target << "WriteProcResTable, "
      << Target << "WriteLatencyTable, "
@@ -2028,7 +2028,7 @@ void SubtargetEmitter::run(raw_ostream &OS) {
   if (NumProcs)
     OS << "ArrayRef(" << Target << "SubTypeKV, " << NumProcs << "), ";
   else
-    OS << "None, ";
+    OS << "std::nullopt, ";
   OS << '\n'; OS.indent(24);
   OS << Target << "WriteProcResTable, "
      << Target << "WriteLatencyTable, "


        


More information about the llvm-commits mailing list