[clang] [clang-repl] Extend the C support. (PR #89804)
Vassil Vassilev via cfe-commits
cfe-commits at lists.llvm.org
Mon May 20 07:38:29 PDT 2024
================
@@ -407,6 +406,16 @@ void IncrementalParser::CleanUpPTU(PartialTranslationUnit &PTU) {
}
}
}
+
+ // FIXME: We should de-allocate MostRecentTU
+ for (Decl *D : MostRecentTU->decls()) {
+ if (!isa<NamedDecl>(D))
+ continue;
+ // Check if we need to clean up the IdResolver chain.
+ NamedDecl *ND = cast<NamedDecl>(D);
----------------
vgvassilev wrote:
OTOH, that loop is easier to be spelled like:
```
for() {
if (auto *ND = dyn_cast<NamedDecl>(D))
// do some work
}
```
That violates the loop early exit policy :)
https://github.com/llvm/llvm-project/pull/89804
More information about the cfe-commits
mailing list