[PATCH] D45383: Limit types of builtins that can be redeclared.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 16 12:25:37 PDT 2018


erichkeane added a comment.

In https://reviews.llvm.org/D45383#1069057, @efriedma wrote:

> I don't see an `A` in `LANGBUILTIN(__va_start,       "vc**.", "nt", ALL_MS_LANGUAGES)`


Ah, you're right!  I was confusing it with LIBBUILTIN(va_start, "vA.", "fn", "stdarg.h", ALL_LANGUAGES) and BUILTIN(__builtin_va_start, "vA.", "nt").

It seems that the MS builtins dont' actually use 'A'.  I'll implement your suggestion, thanks for your patience :)


https://reviews.llvm.org/D45383





More information about the cfe-commits mailing list