[PATCH] D33644: Add default values for function parameter chunks
Manuel Klimek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 12 07:06:21 PDT 2017
klimek added a reviewer: rsmith.
klimek added a comment.
+Richard, as apparently we get the source ranges for default values of built-in types without the preceding "=", but for user-defined types including the preceding "=" - is that intentional?
================
Comment at: lib/Sema/SemaCodeComplete.cpp:2424
+ // If we don't have '=' in front of value.
+ // Lexer returns built-in types values without '=' and user-defined types values with it.
+ return " = " + DefValue;
----------------
"The Lexer returns default-values of built-in types without '=', and default-value of user-defined types with it."
But apart from that: that seems like a bug in the range of the default value? I'm now really confused :)
https://reviews.llvm.org/D33644
More information about the cfe-commits
mailing list