https://github.com/sdkrystian closed
https://github.com/llvm/llvm-project/pull/81171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sdkrystian wrote:
@erichkeane I'm building the check-clang-extra target locally just to make sure
that the test passes... should be done in ~5 mins
https://github.com/llvm/llvm-project/pull/81171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/PiotrZSL approved this pull request.
LGTM, purpose of this test is just to verify that errors (any) are shown in
clang-tidy
https://github.com/llvm/llvm-project/pull/81171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/erichkeane approved this pull request.
This seems right to me, though a 2nd reviewer would be nice. I don't think we
should wait for CI on this due to it fixing a bot-breakage.
https://github.com/llvm/llvm-project/pull/81171
___
cf
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/81171
>From ca871fca01e07bd1dcbdab3256cb1631dabca853 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Thu, 8 Feb 2024 13:42:38 -0500
Subject: [PATCH] [clang-tidy] Fix failing test after
17f0680f69f44d340fd0
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Krystian Stasiowski (sdkrystian)
Changes
The following test case in
`clang-tools-extra/test/clang-tidy/infrastructure/diagnostic.cpp` is failing:
```cpp
#ifdef PR64602 // Should not crash
template
struct S
{
https://github.com/sdkrystian edited
https://github.com/llvm/llvm-project/pull/81171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
