fixup! [Clang][HIP] Deprecate the AMDGCN_WAVEFRONT_SIZE macros

use plain Twine with empty string instead of optional

Author: ritter-x2a

clang/include/clang/Basic/MacroBuilder.h

@@ -17,7 +17,6 @@
 #include "clang/Basic/LLVM.h"
 #include "llvm/ADT/Twine.h"
 #include "llvm/Support/raw_ostream.h"
-#include <optional>
 
 namespace clang {
 
@@ -30,11 +29,11 @@ class MacroBuilder {
   /// If DeprecationMsg is provided, also append a pragma to deprecate the
   /// defined macro.
   void defineMacro(const Twine &Name, const Twine &Value = "1",
-                   std::optional<Twine> DeprecationMsg = std::nullopt) {
+                   Twine DeprecationMsg = "") {
     Out << "#define " << Name << ' ' << Value << '\n';
-    if (DeprecationMsg.has_value())
-      Out << "#pragma clang deprecated(" << Name << ", \""
-          << DeprecationMsg.value() << "\")\n";
+    if (!DeprecationMsg.isTriviallyEmpty())
+      Out << "#pragma clang deprecated(" << Name << ", \"" << DeprecationMsg
+          << "\")\n";
   }
 
   /// Append a \#undef line for Name.  Name should be of the form XXX