[clang] [HLSL] Add validation for the -enable-16bit-types option (PR #85340)

Chris B via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 21 16:44:27 PDT 2024


================
@@ -4258,6 +4258,18 @@ bool CompilerInvocation::ParseLangArgs(LangOptions &Opts, ArgList &Args,
       } else {
         llvm_unreachable("expected DXIL or SPIR-V target");
       }
+      // validate that if fnative-half-type is given, that
+      // the language standard is at least hlsl2021, and that
+      // the target shader model is at least 6.2
+      if (Args.getLastArg(OPT_fnative_half_type)) {
----------------
llvm-beanz wrote:

We currently support fp16 in Vulkan 1.0, so this check should probably only apply to shader model targets:

https://godbolt.org/z/Wefc6179b

(@sudonatalie or @Keenuts can confirm)

https://github.com/llvm/llvm-project/pull/85340


More information about the cfe-commits mailing list