[clang] [llvm] [HLSL] AST support for WaveSize attribute. (PR #101240)

Chris B via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 30 17:25:44 PDT 2024


================
@@ -2862,6 +2862,10 @@ static bool mergeDeclAttribute(Sema &S, NamedDecl *D,
   else if (const auto *NT = dyn_cast<HLSLNumThreadsAttr>(Attr))
     NewAttr = S.HLSL().mergeNumThreadsAttr(D, *NT, NT->getX(), NT->getY(),
                                            NT->getZ());
+  else if (const auto *NT = dyn_cast<HLSLWaveSizeAttr>(Attr))
----------------
llvm-beanz wrote:

This code block looks ripe for a refactoring to use a switch to detect the attribute type... We should consider doing that cleanup in a follow-up change.

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


More information about the cfe-commits mailing list