================ @@ -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