[PATCH] D59121: [analyzer] Fix macro names in diagnostics within bigger macros.

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 29 14:30:31 PDT 2019


Charusso added a comment.

In D59121#1448386 <https://reviews.llvm.org/D59121#1448386>, @NoQ wrote:

> Mmm, that *is* an `Assuming...` piece, i.e., this is the same code, just the structure of macros is more complicated than usual.


You told me we would like to see a value when we hover over a name, which is cool. If I think about C and they do not C# over the complicated macros, I would like to C names of macros. It is up to you, but your first intention working with my code very well.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59121/new/

https://reviews.llvm.org/D59121





More information about the cfe-commits mailing list