rupprecht added a comment. This is causing a test failure: https://buildkite.com/llvm-project/upstream-bazel/builds/48607#0185190c-43f8-43ff-b8bd-fa8ce0b6e2f5 (and likewise running `ninja check-clang-unit` locally, but I don't have a buildbot link to that)
================ Comment at: clang/lib/Format/Format.cpp:3408 Passes.emplace_back([&](const Environment &Env) { - return BracesInserter(Env, Expanded).process(/*SkipAnnotation=*/true); + return BracesInserter(Env, S).process(/*SkipAnnotation=*/true); }); ---------------- This looks like use-after-free. S is captured by reference but the body of the lambda executes after this scope block has ended. BracesInserter itself takes it by ref but then stores a copy, but by the time the constructor runs, the reference is already dangling. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140058/new/ https://reviews.llvm.org/D140058 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits