https://github.com/jansvoboda11 edited
https://github.com/llvm/llvm-project/pull/67839
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/67839
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/67839
>From 9c798ed914b0008d98587c94f8ee3bb914412215 Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 8 Sep 2023 16:39:10 -0700
Subject: [PATCH 1/3] [clang][tidy] Ensure rewriter has the correct CWD
This
@@ -189,9 +191,12 @@ class ErrorReporter {
void finish() {
if (TotalFixes > 0) {
- Rewriter Rewrite(SourceMgr, LangOpts);
+ bool AnyNotWritten = false;
for (const auto : FileReplacements) {
+Rewriter Rewrite(SourceMgr, LangOpts);
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/67839
>From 9c798ed914b0008d98587c94f8ee3bb914412215 Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 8 Sep 2023 16:39:10 -0700
Subject: [PATCH 1/2] [clang][tidy] Ensure rewriter has the correct CWD
This
jansvoboda11 wrote:
Ping @PiotrZSL.
https://github.com/llvm/llvm-project/pull/67839
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jansvoboda11 wrote:
> Clang part looks fine. For a clang-tidy part, is there a way to test this
> part ? What changes because we use now a absolute build directory. I'm
> missing some tests for that part.
So the clang-tidy part is necessary to keep the
https://github.com/PiotrZSL approved this pull request.
Clang part looks fine.
For a clang-tidy part, is there a way to test this part ? What changes because
we use now a absolute build directory.
I'm missing some tests for that part.
https://github.com/llvm/llvm-project/pull/67839