rjmccall added inline comments.

================
Comment at: include/clang/Basic/AttrDocs.td:2371
@@ +2370,3 @@
+
+Not all targets support this attribute.  ELF targets support this attribute 
when using binutils v2.20.1 or higher and glibc v2.11.1 or higher.  Non-ELF 
targets currently do not support this attribute.
+  }];
----------------
echristo wrote:
> Probably better to say linux fwiw and not ELF.
The validation code in Sema is checking for an ELF target.  If the restriction 
is more precise than that, then we should make a TargetInfo callback.  Do the 
BSDs and other ELF targets not use binutils/glibc?


http://reviews.llvm.org/D15524



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

Reply via email to