[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-13 Thread Mariya Podchishchaeva via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbba6ea8c2d56: [C++20] Claim full support for consteval again (authored by Fznamznon). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-13 Thread Mariya Podchishchaeva via Phabricator via cfe-commits
Fznamznon added a comment. The CI failure is unrelated and I suppose should be already fixed by aa28875 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147717/new/ https://review

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-12 Thread Mariya Podchishchaeva via Phabricator via cfe-commits
Fznamznon updated this revision to Diff 530477. Fznamznon added a comment. Rebase, add a release note Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147717/new/ https://reviews.llvm.org/D147717 Files: clang/docs/ReleaseNotes.rst clang/lib/Front

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-12 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D147717#4413593 , @Fznamznon wrote: > In D147717#4413522 , @aaron.ballman > wrote: > >> LGTM (though I wonder if we should add a release note as well) > > I can add a note saying

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-12 Thread Mariya Podchishchaeva via Phabricator via cfe-commits
Fznamznon added a comment. In D147717#4413522 , @aaron.ballman wrote: > LGTM (though I wonder if we should add a release note as well) I can add a note saying that the macro is now defined. Is somewhere under `C++ Language Changes` title the right plac

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-12 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM (though I wonder if we should add a release note as well) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147717/new/ https://reviews.llvm.org/D147717

[PATCH] D147717: [C++20] Claim full support for consteval again

2023-06-12 Thread Mariya Podchishchaeva via Phabricator via cfe-commits
Fznamznon added a comment. https://github.com/llvm/llvm-project/issues/62886 has been fixed, I'm trying to figure out or at least reduce https://github.com/llvm/llvm-project/issues/60709 so it doesn't have headers included, but no progress so far. Repository: rG LLVM Github Monorepo CHANGES