swiftix added inline comments.

================
Comment at: include/clang/Basic/Attr.td:1394
@@ -1393,1 +1393,3 @@
 
+def PreserveMost : InheritableAttr {
+  let Spellings = [GNU<"preserve_most">];
----------------
aaron.ballman wrote:
> Do these attributes do anything on targets other than AArch64 and x86-64? If 
> not, these should probably be inheriting from TargetSpecificAttr. (With tests 
> confirming the behavior on other targets.)
These attributes are not target-specific, IMHO, and they would make sense for 
other targets as well.  But they are currently implemented only for AArch64 and 
x86-64.


http://reviews.llvm.org/D18025



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

Reply via email to