[clang-tools-extra] [clangd] Consider expression statements in ExtractVariable tweak (PR #112525)

Christian Kandeler via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 10 04:42:16 PST 2024


================
@@ -423,8 +427,6 @@ bool childExprIsStmt(const Stmt *Outer, const Expr *Inner) {
   if (!Outer || !Inner)
     return false;
   // Exclude the most common places where an expr can appear but be unused.
-  if (llvm::isa<CompoundStmt>(Outer))
----------------
ckandeler wrote:

I made the comment rather generic, as I found it difficult to summarize what the function does without essentially repeating the code.

https://github.com/llvm/llvm-project/pull/112525


More information about the cfe-commits mailing list