[PATCH] D33644: Add default values for function parameter chunks

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 28 03:38:06 PDT 2017


klimek added a comment.

In https://reviews.llvm.org/D33644#783903, @yvvan wrote:

> Do not evaluate numbers.
>  Check for != "=" is needed not to mess with invalid default arguments or their types (without it I get "const Bar& bar = =" when Bar is not defined)


Shouldn't we than instead check that error case?


https://reviews.llvm.org/D33644





More information about the cfe-commits mailing list