[PATCH] D59560: Permit redeclarations of a builtin to specify calling convention.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 19 14:18:09 PDT 2019


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


================
Comment at: lib/Sema/SemaDecl.cpp:3144
+          << FunctionType::getNameForCallConv(NewTypeInfo.getCC())
+          << 3 /*on builtin function*/;
+      NewTypeInfo = NewTypeInfo.withCallingConv(OldTypeInfo.getCC());
----------------
rnk wrote:
> 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.
Ah, great!  I didn't realize there was a convention for how to do these.


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

https://reviews.llvm.org/D59560





More information about the cfe-commits mailing list