[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Aaron Ballman via cfe-commits
AaronBallman wrote: > > Can you do the JSONNodeDumper as well so the two stay somewhat in-sync? > > Looks like that goes through `APValue::printPretty()`, so nothing to do for > that. Ah, good to know, thanks! https://github.com/llvm/llvm-project/pull/136130 __

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/136130 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman approved this pull request. LGTM assuming precommit CI is happy too. https://github.com/llvm/llvm-project/pull/136130 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Timm Baeder via cfe-commits
tbaederr wrote: > Can you do the JSONNodeDumper as well so the two stay somewhat in-sync? Looks like that goes through `APValue::printPretty()`, so nothing to do for that. https://github.com/llvm/llvm-project/pull/136130 ___ cfe-commits mailing list

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman commented: Can you do the JSONNodeDumper as well so the two stay somewhat in-sync? https://github.com/llvm/llvm-project/pull/136130 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/m

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes Print the member pointer decl and the path. --- Full diff: https://github.com/llvm/llvm-project/pull/136130.diff 3 Files Affected: - (modified) clang/lib/AST/TextNodeDumper.cpp (+14-2) - (modified) clang/

[clang] [clang] Implement dump() for MemberPointer APValues (PR #136130)

2025-04-17 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/136130 Print the member pointer decl and the path. >From ae8fac6c80a757a5ee3a0fa6e59d7546f71ce116 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Thu, 17 Apr 2025 13:44:50 +0200 Subject: [PATCH] [c