[PATCH] D67159: [clang] New __attribute__((__clang_arm_mve_alias)).

[Simon Tatham via Phabricator via cfe-commits]

simon_tatham marked an inline comment as done.
simon_tatham added inline comments.


================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7442
+  case ParsedAttr::AT_ClangBuiltinOverride:
+    handleClangBuiltinOverrideAttribute(S, D, AL);
+    break;
----------------
aaron.ballman wrote:
> You should be able to call `handleSimpleAttribute<ClangBuiltinOverrideAttr>()` instead.
Oh, nearly forgot: apparently I can't, because that template expects the attribute to have a constructor that takes only an `ASTContext` and an `AttributeCommonInfo`. But the constructor for my attribute also takes an `IdentifierInfo` giving the builtin name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67159/new/

https://reviews.llvm.org/D67159