================
@@ -46,13 +46,32 @@ class CodeGenOptionsBase {
   enum class CompatibilityKind {
     /// Does affect the construction of the AST in a way that does prevent
     /// module interoperability.
-    Affecting,
+    NotCompatible,
+    /// Does affect the construction of the AST in a way that doesn't prevent
+    /// interoperability (that is, the value can be different between an
+    /// explicit module and the user of that module).
+    Compatible,
     /// Does not affect the construction of the AST in any way (that is, the
     /// value can be different between an implicit module and the user of that
     /// module).
     Benign,
   };
 
+  using CFBranchLabelSchemeKind = CFBranchLabelSchemeKind;
+  using ProfileInstrKind = llvm::driver::ProfileInstrKind;
----------------
jansvoboda11 wrote:

These are necessary, because now we use these types outside of `CodeGenOptions` 
in `ASTReader::ParseCodeGenOptions()` so we either need to fully qualify the 
types in `CodeGenOptions.def` or leave just the type names without namespaces, 
pull the types into `CodeGenOptions` and qualify them with 
`clang::CodeGenOptions::` in clients. That's what `LangOptions` do, so I chose 
to do it here for consistency.

https://github.com/llvm/llvm-project/pull/146422
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to