[PATCH] D67159: [clang] New __attribute__((__clang_arm_mve_alias)).
Simon Tatham via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 2 03:17:36 PDT 2019
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
More information about the cfe-commits
mailing list