[PATCH] D44646: Sema: in msvc compatibility mode, don't allow forceinline on variadics
Dustin L. Howett via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 19 15:02:21 PDT 2018
DHowett-MSFT added a comment.
In https://reviews.llvm.org/D44646#1042347, @efriedma wrote:
> The compiler shouldn't inline functions which call va_start, whether or not they're marked always_inline. That should work correctly, I think, at least on trunk. (See https://reviews.llvm.org/D42556 .)
>
> If you want to warn anyway, that's okay.
Interesting. I certainly hit the lowering/instruction scheduling crash after that change landed. Perhaps it's a valuable thing for me to file a bug on.
Repository:
rC Clang
https://reviews.llvm.org/D44646
More information about the cfe-commits
mailing list