[PATCH] D45383: Strip reference from a va_list object in C when merging parameter types.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 6 13:06:54 PDT 2018


erichkeane added a comment.

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

> Rather than adding weird hacks to merging, can we just reject any attempt to redeclare a builtin?


I don't believe we can.  I think things often re-declare them, particularly when it comes to va_end.  I'd love to move the 'builtinID' check higher and reject any attempts to redeclare in C, but I'd like some further guidance.  Do you know of anyone who can help us make that decision?


Repository:
  rC Clang

https://reviews.llvm.org/D45383





More information about the cfe-commits mailing list