[PATCH] D59560: Permit redeclarations of a builtin to specify calling convention.
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 19 14:14:45 PDT 2019
rnk added inline comments.
================
Comment at: lib/Sema/SemaDecl.cpp:3144
+ << FunctionType::getNameForCallConv(NewTypeInfo.getCC())
+ << 3 /*on builtin function*/;
+ NewTypeInfo = NewTypeInfo.withCallingConv(OldTypeInfo.getCC());
----------------
You can make these self-documenting with an enum, and then reference the enum name in a comment in the .td file. It seems the existing convention is to put these types of enums directly into the Sema class.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59560/new/
https://reviews.llvm.org/D59560
More information about the cfe-commits
mailing list