arichardson marked 2 inline comments as done.
arichardson 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()))
----------------
aaron.ballman wrote:
> 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?
I added the test to  D86721 since this change handles __attribute__ and that 
one deals with [[attr]]


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

Reply via email to