[PATCH] D86135: [analyzer][MacroExpansion] Fix a crash where multiple parameters resolved to __VA_ARGS__

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 7 08:33:10 PDT 2020


Szelethus added a comment.

In D86135#2259325 <https://reviews.llvm.org/D86135#2259325>, @steakhal wrote:

> Perfectly clear, thank you. However, I would still rely on the others to accept this :|
>
> BTW why does the `plist-macros-with-expansion.cpp.plist` change? It makes the diff somewhat noisy :s

Well, I removed a line, so every other entry about file position is changed in the plist file. I think I could just remove the entire thing altogether, its not like the actual plist output is what we're looking for, at least not in its many-thousand-line entirety.


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

https://reviews.llvm.org/D86135



More information about the cfe-commits mailing list