================
@@ -740,7 +740,8 @@ bool tools::isTLSDESCEnabled(const ToolChain &TC,
     SupportedArgument = V == "desc" || V == "trad";
     EnableTLSDESC = V == "desc";
   } else if (Triple.isX86()) {
-    SupportedArgument = V == "gnu";
+    SupportedArgument = V == "gnu" || V == "gnu2";
+    EnableTLSDESC = V == "gnu2";
----------------
ilovepi wrote:

Is it possible to make this option work uniformly with `trad` and `desc` too?  
The `gnu`/`gnu2` spelling is something we need to support for compatibility, 
but it would be good if we can get this to be consistent in clang. I'm hoping 
we can get GCC to accept the same set of options.

@MaskRay does that sound reasonable to you?

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

Reply via email to