]> granicus.if.org Git - clang/commit
Now that macro expansion notes are real notes and go through the same
authorChandler Carruth <chandlerc@gmail.com>
Sun, 16 Oct 2011 09:30:08 +0000 (09:30 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Sun, 16 Oct 2011 09:30:08 +0000 (09:30 +0000)
commita47129e97be2379961046fac4f94f12ac15cb540
tree0124269a49e79068a3cb3d800c5531bc7c8263a5
parent938ef249e9e5b5e3b7065c7f92b2578aa2dd971b
Now that macro expansion notes are real notes and go through the same
formatting as any other diagnostic, they will be properly line wrapped and
otherwise pretty printed. Let's take advantage of that and the new factoring to
add some helpful information to them (much like template backtrace notes and
other notes): the name of the macro whose expansion is being noted. This makes
a world of difference if caret diagnostics are disabled, making the expansion
notes actually useful in this case. It also helps ensure that in edge cases the
information the user needs is present. Consider:

% nl -ba t5.cc
     1  #define M(x, y, z) \
     2    y
     3
     4  M(
     5    1,
     6    2,
     7    3);

We now produce:
% ./bin/clang -fsyntax-only t5.cc
t5.cc:6:3: error: expected unqualified-id
  2,
  ^
t5.cc:2:3: note: expanded from macro: M
  y
  ^
1 error generated.

Without the added information in the note, the name of the macro being expanded
would never be shown.

This also deletes a FIXME to use the diagnostic formatting. It's not yet clear
to me that we *can* do this reasonably, and the production of this message was
my primary goal here anyways.

I'd love any comments or suggestions on improving these notes, their wording,
etc. Currently, I need to make them provide more helpful information in the
presence of a token-pasting buffer, and I'm pondering adding something along
the lines of "expanded from argument N of macro: ...".

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@142127 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Frontend/TextDiagnostic.cpp
test/Misc/caret-diags-macros.c
test/Misc/include-stack-for-note-flag.cpp
test/Misc/macro-backtrace.c