[PATCH] D100046: [AArch64] ACLE: Fix issue for mismatching enum types with builtins.

[Raul Tambre via Phabricator via cfe-commits]

tambre added inline comments.


================
Comment at: clang/lib/Sema/SemaDecl.cpp:10953-10967
     if (unsigned BuiltinID = NewFD->getBuiltinID()) {
       ASTContext::GetBuiltinTypeError Error;
       LookupNecessaryTypesForBuiltin(S, BuiltinID);
       QualType T = Context.GetBuiltinType(BuiltinID, Error);
       // If the type of the builtin differs only in its exception
       // specification, that's OK.
       // FIXME: If the types do differ in this way, it would be better to
----------------
This whole block is unnecessary since D77491 and should've been removed, but seems I missed it. Reverting/forgetting builtins was problematic and the attribute-based approach solved those deficiencies.

Delete this whole block and also remove `forgetBuiltin()`. The builtin-related code in `ActOnFunctionDeclarator()` handles everything necessary. No tests break and the SVE ones will pass. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100046