================
@@ -1033,12 +1033,15 @@ void SemaHLSL::handleRootSignatureAttr(Decl *D, const 
ParsedAttr &AL) {
 void SemaHLSL::handleNumThreadsAttr(Decl *D, const ParsedAttr &AL) {
   llvm::VersionTuple SMVersion =
       getASTContext().getTargetInfo().getTriple().getOSVersion();
+  bool IsDXIL = getASTContext().getTargetInfo().getTriple().getArch() ==
+                llvm::Triple::dxil;
+
   uint32_t ZMax = 1024;
----------------
s-perron wrote:

I don't think so. These are close to the limits on actual machines. For 
Android, the existing checks are way beyond the limit.

See maxComputeWorkGroupInvocations and  maxComputeWorkGroupSize in 
https://vulkan.lunarg.com/doc/sdk/1.4.313.2/windows/profiles_definitions.html.

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

Reply via email to