[patch] Delay __fastcall attribute checking utnil after merging declarations

Reid Kleckner rnk at google.com
Thu Aug 21 11:27:25 PDT 2014

On Tue, Aug 19, 2014 at 10:40 AM, Nico Weber <thakis at chromium.org> wrote:
> I did give it a try, and rejecting stdcall fires on Windows SDK headers:
> ...win8sdk/Include/um\imm.h(432,13) :  error(clang): function with no
> prototype cannot use stdcall calling convention
> BOOL WINAPI ImmDisableLegacyIME();
>             ^

Bummer. :( We could make it a default error warning, thereby suppressing it
in system headers, I guess.

> So the same mechanism doesn't work for stdcall. We could do the same thing
> for thiscall, but if msvc rejects that in C files, we should probably just
> do that too?

GCC allows __attribute__((thiscall)) in C mode. I don't think it's worth
the complexity of diagnosing this differently in different modes.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140821/dfe14861/attachment.html>

More information about the cfe-commits mailing list