aaron.ballman added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1895
                            tok::kw_restrict, tok::kw_volatile,
-                           tok::kw_noexcept) ||
+                           tok::kw___attribute, tok::kw_noexcept) ||
         (NextToken->is(tok::l_brace) && !NextToken->getNextNonComment()))
----------------
arichardson wrote:
> arichardson wrote:
> > aaron.ballman wrote:
> > > What about other attributes than GNU-style ones, like 
> > > `[[clang::address_space(0)]]`?
> > I guess those should also be handled. I can try to do that in a follow-up 
> > change.
> It appears that they are already handled for this case. I've handled them in 
> cast expressions in D86721. 
Ah, thank you for the update! Can you add a test case for that, as I don't see 
one in the current tests?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86711/new/

https://reviews.llvm.org/D86711

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D86711: [clan... Aaron Ballman via Phabricator via cfe-commits

Reply via email to