[llvm-commits] [PATCH 1/2] EE/Interpreter/ExternalFunctions.cpp: Prune "C" linkage to suppress warnings with -Wreturn-type (and MSC's w4190).

Chris Lattner clattner at apple.com
Thu Feb 23 15:36:40 PST 2012


On Feb 11, 2012, at 5:22 PM, NAKAMURA Takumi wrote:

> In historical reason, Interpreter's external entries had prefix
> "lle_X_" as C linkage, even for well-known entries in EE/Interpreter.
> Now, at least on ToT, they are resolved via FuncNames[] mapper.
> We will not need their symbols are expected to be exported any more.
> 
> Clang r150128 has introduced the warning <"%0 has C-linkage specified,
> but returns user-defined type %1 which is incompatible with C">.

These look fine to me if they haven't been applied already,

-Chris




More information about the llvm-commits mailing list