https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/91071
___
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.
https://github.com/llvm/llvm-project/pull/91071
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mikecrowe wrote:
@5chmidti wrote:
> LGTM, thanks
Please can you land this for me since I don't have permission to do so myself?
(Assuming you do have permission.)
Thanks.
https://github.com/llvm/llvm-project/pull/91071
___
cfe-commits mailing list
https://github.com/mikecrowe updated
https://github.com/llvm/llvm-project/pull/91071
>From d25fa679a70947d42fe3125858741508475d67c0 Mon Sep 17 00:00:00 2001
From: Mike Crowe
Date: Sat, 4 May 2024 17:41:37 +0100
Subject: [PATCH] [clang-tidy] Improve modernize-use-std-print diagnostic
Include
https://github.com/5chmidti approved this pull request.
LGTM, thanks
https://github.com/llvm/llvm-project/pull/91071
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mikecrowe wrote:
As suggested by @5chmidti in
https://github.com/llvm/llvm-project/pull/90397/files/54c325d7a5e24441adbe8036800a2f50e2ff5fa0#r1589445886
.
https://github.com/llvm/llvm-project/pull/91071
___
cfe-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Mike Crowe (mikecrowe)
Changes
Include the source range of the printf function in the diagnostic so it gets
underlined in the output.
---
Full diff: https://github.com/llvm/llvm-project/pull/91071.diff
1 Files Affected:
-
https://github.com/mikecrowe created
https://github.com/llvm/llvm-project/pull/91071
Include the source range of the printf function in the diagnostic so it gets
underlined in the output.
>From d0ccc6ade357155161977446b64c50cafa100edb Mon Sep 17 00:00:00 2001
From: Mike Crowe
Date: Sat, 4