[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-04-05 Thread via cfe-commits
yronglin wrote: Thanks working on this! Can you add some description for this patch? https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-29 Thread Fangrui Song via cfe-commits
MaskRay wrote: LGTM! Thanks for the fix https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-28 Thread Qiu Chaofan via cfe-commits
ecnelises wrote: Done, thanks! https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-28 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises closed https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-27 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises edited https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-26 Thread via cfe-commits
https://github.com/cor3ntin approved this pull request. LGTM. Will you need me to merge this for you? https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listin

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-25 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises updated https://github.com/llvm/llvm-project/pull/131608 >From 60b1e16b7b297095a50ed8db644b787e35f8eab0 Mon Sep 17 00:00:00 2001 From: Qiu Chaofan Date: Mon, 17 Mar 2025 20:49:02 +0800 Subject: [PATCH 1/2] [Clang] Check PP presence when printing stats --- clang/li

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-25 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises edited https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-25 Thread via cfe-commits
cor3ntin wrote: This change needs a release note. Please add an entry to `clang/docs/ReleaseNotes.rst` in the section the most adapted to the change, and referencing any Github issue this change fixes. Thanks! https://github.com/llvm/llvm-project/pull/131608 ___

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-24 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises updated https://github.com/llvm/llvm-project/pull/131608 >From 60b1e16b7b297095a50ed8db644b787e35f8eab0 Mon Sep 17 00:00:00 2001 From: Qiu Chaofan Date: Mon, 17 Mar 2025 20:49:02 +0800 Subject: [PATCH] [Clang] Check PP presence when printing stats --- clang/lib/Fr

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-24 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises edited https://github.com/llvm/llvm-project/pull/131608 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-22 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik commented: Thank you for the fix! Second, that we need a summary explaining the problem and solution. Is this linked to a bug report, if it is, then we should mention that bug report as well. I would also like to see a test showing a case where we successfully print

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-17 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Qiu Chaofan (ecnelises) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/131608.diff 2 Files Affected: - (modified) clang/lib/Frontend/FrontendAction.cpp (+8-4) - (added) clang/test/Frontend/print-stats.ll (+6)

[clang] [Clang] Check PP presence when printing stats (PR #131608)

2025-03-17 Thread Qiu Chaofan via cfe-commits
https://github.com/ecnelises created https://github.com/llvm/llvm-project/pull/131608 None >From c7d48ef991e3dbd798d880d846127a393c395d4e Mon Sep 17 00:00:00 2001 From: Qiu Chaofan Date: Mon, 17 Mar 2025 20:49:02 +0800 Subject: [PATCH] [Clang] Check PP presence when printing stats --- clang/