[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