================
@@ -407,7 +407,8 @@ def MicrosoftExt : LangOpt<"MicrosoftExt">;
 def Borland : LangOpt<"Borland">;
 def CUDA : LangOpt<"CUDA">;
 def HIP : LangOpt<"HIP">;
-def SYCL : LangOpt<"SYCLIsDevice">;
+def SYCLHost : LangOpt<"SYCLIsHost">;
----------------
erichkeane wrote:

Is the `Is` well motivated in these lang-opts?  I realize `historical reasons`, 
but i find myself questioning the naming here.  `SYCLDevice` and `SYCLHost` is 
perhaps fine.  Also-also, I find myself thinking that to match the coding 
standard, these should use `Sycl`.  IDK why CUDA doesn't follow the rule (I 
think only 3 letter acronyms can be capital?).

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

Reply via email to