https://github.com/efriedma-quic commented:
Instead of creating a separate cleanup stack and adding special deferred handling for that cleanup stack, can we just skip creating the scope in the first place? We have `LexicalScope ForScope(*this, S.getSourceRange());`; can we replace that with something like the following? ``` std::optional<LexicalScope> ForScope; if (getLangOpts().C99 || getLangOpts().CPlusPlus) ForScope.emplace(*this, S.getSourceRange()); ``` https://github.com/llvm/llvm-project/pull/156643 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
