krasimir added inline comments.

================
Comment at: unittests/Format/FormatTest.cpp:6068-6069
+  verifyFormat("void f() [[deprecated(\"so sorry\")]];");
+  verifyFormat("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa\n"
+               "    [[unused]] aaaaaaaaaaaaaaaaaaaaaaa(int i);");
+}
----------------
benhamilton wrote:
> I couldn't figure out why this breaks before the `[` in `[[unused]]`, when 
> the identical test using `__attribute__((unused))` above does *not* break 
> before the `__attribute__`.
> 
> I ran with `-debug` and the two seem fairly similar. Can anyone help shed 
> light on this?
> 
> ## With `__attribute__((unused))`
> 
> P8067
> 
> ## With `[[unused]]`
> 
> P8068
The block of `a`-s appears to be longer here than in the `__attribute__` 
example. That might be a reason.


Repository:
  rC Clang

https://reviews.llvm.org/D43902



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to