[PATCH] D52795: [analyzer][PlistMacroExpansion] Part 3.: Macro arguments are expanded

Whisperity via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 6 03:16:16 PST 2018

whisperity added inline comments.

Comment at: lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:1018
+    auto It = CurrExpArgTokens.begin();
+    while (It != CurrExpArgTokens.end()) {
+      if (It->isNot(tok::identifier)) {
xazax.hun wrote:
> Maybe a for loop mor natural here?
I asked this one already in the earlier (non-split) diff and the reason for the `while` is that there are a lot of conditional moves, advancements and even an `erase` call in the loop body.


More information about the cfe-commits mailing list