[PATCH] D151033: [clang][AST] TextNodeDumper should not evaluate the initializer of constexpr variable template definition

2023-05-21 Thread Takuya Shimizu via Phabricator via cfe-commits
hazohelet added a comment. In D151033#4359121 , @tbaeder wrote: > Can you briefly explain why the `FieldDecl` we see is null in that case? Consider `template constexpr T foo{};` `evaluateValue` here leads to a call of `EvaluateAsInitializer`. It calls

[PATCH] D151033: [clang][AST] TextNodeDumper should not evaluate the initializer of constexpr variable template definition

2023-05-21 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. Can you briefly explain why the `FieldDecl` we see is null in that case? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151033/new/ https://reviews.llvm.org/D151033 ___ cfe-commit

[PATCH] D151033: [clang][AST] TextNodeDumper should not evaluate the initializer of constexpr variable template definition

2023-05-20 Thread Takuya Shimizu via Phabricator via cfe-commits
hazohelet created this revision. hazohelet added reviewers: aaron.ballman, erichkeane, tbaeder. Herald added a project: All. hazohelet requested review of this revision. Herald added a project: clang. `TextNodeDumper` enabed through `-ast-dump` flag should not evlauate the initializer when it vis