arichardson added a comment. In D86713#2242253 <https://reviews.llvm.org/D86713#2242253>, @JakeMerdichAMD wrote:
> LGTM, again assuming tests pass locally (patch did not resolve). > > Out of curiosity, is _Atomic on your radar? I found some code in clang proper > that handled restrict and _Atomic together. C/C++ have way too many > qualifiers... I have not looked at _Atomic yet and it's probably low priority for me unless it's a trivial change. My main motivation with these changes is to format a `__capability` pointer qualifier correctly (an extension that we add for our out-of-tree CHERI C/C++ dialect). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86713/new/ https://reviews.llvm.org/D86713 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
