[Lldb-commits] [lldb] a1c0ba1 - [lldb][test] TestDataFormatterLibcxxOptionalSimulator.py: change order of ifdefs

Michael Buch via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 7 03:12:35 PDT 2024


Author: Michael Buch
Date: 2024-10-07T11:12:01+01:00
New Revision: a1c0ba1646f5a80dadd34f55e4a36d5409c2675a

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

LOG: [lldb][test] TestDataFormatterLibcxxOptionalSimulator.py: change order of ifdefs

The current layout *does* have `removecv_t`. So change
the ifdefs to reflect that.

Added: 
    

Modified: 
    lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/optional/main.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/optional/main.cpp b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/optional/main.cpp
index ee4763f0b22c40..94f3c75d395f86 100644
--- a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/optional/main.cpp
+++ b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/optional/main.cpp
@@ -3,7 +3,7 @@
 
 #if REVISION == 0
 // Pre-a3942b3 layout.
-#define HAS_REMOVE_CV
+#define NO_REMOVE_CV
 #endif
 // REVISION == 1: current layout
 
@@ -20,10 +20,10 @@ struct __optional_destruct_base {
   typedef _Tp value_type;
   union {
     char __null_state_;
-#ifdef HAS_REMOVE_CV
-    remove_cv_t<value_type> __val_;
-#else // !HAS_REMOVE_CV
+#ifdef NO_REMOVE_CV
     value_type __val_;
+#else // !NO_REMOVE_CV
+    remove_cv_t<value_type> __val_;
 #endif
   };
   bool __engaged_;


        


More information about the lldb-commits mailing list