[clang] [Utils] add update-verify-tests.py (PR #97369)

hnrklssn wrote: Very sorry about the CI oversight on my part! Thanks for reverting, I was in a meeting. Looks like I was bitten by Windows line endings again... https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@l

[clang] [Utils] add update-verify-tests.py (PR #97369)

jurahul wrote: Reverted in https://github.com/llvm/llvm-project/pull/108630 https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Utils] add update-verify-tests.py (PR #97369)

jurahul wrote: Also note that this was also detected by CI prior to merging the PR. https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Utils] add update-verify-tests.py (PR #97369)

nico wrote: This breaks tests on windows: http://45.33.8.238/win/94083/step_6.txt Please take a look and revert for now if it takes a while to fix. https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [Utils] add update-verify-tests.py (PR #97369)

jurahul wrote: Looks like these tests are failing in CI: see https://buildkite.com/llvm-project/github-pull-requests/builds/100644#0191ec9f-441f-4299-ad47-a9318fbae7a8 https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-c

[clang] [Utils] add update-verify-tests.py (PR #97369)

https://github.com/hnrklssn closed https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Utils] add update-verify-tests.py (PR #97369)

https://github.com/nhaehnle approved this pull request. I didn't go over the core.py implementation with a comb, but this looks reasonable to me. Thanks! https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.ll

[clang] [Utils] add update-verify-tests.py (PR #97369)

@@ -0,0 +1,10 @@ +// RUN: %clang_cc1 -verify %s +// RUN: diff %s %s.expected nhaehnle wrote: Right, it's all a little mind-bending, but my point is that since you're trying to test whether the invocation of `not %{lit} %S/LitTests --update-tests` in `lit-plugin

[clang] [Utils] add update-verify-tests.py (PR #97369)

https://github.com/hnrklssn edited https://github.com/llvm/llvm-project/pull/97369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits