aaron.ballman added inline comments.
================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:259
@@ +258,3 @@
+def err_anyx86_interrupt_attribute : Error<
+ "interrupt service routine %select{must have 'void' return value|"
+ "can only have a pointer argument and an optional integer argument|"
----------------
I still dislike "interrupt service routine" -- there is no interrupt service
routine involved here because the attribute does not apply to that function to
make it one. The issue is with the attribute's requirements on the function
type, so I still prefer "x86 |x86-64'interrupt' attribute only applies to
functions that...". Also, "have 'void' return value" is still incorrect -- they
need to have a void return type. Something like:
"%select{x86|x86-64}0 'interrupt' attribute only applies to functions that have
%select{a 'void' return type|only a pointer parameter optionally followed by an
integer parameter|a pointer as the first parameter|a %2 type as the second
parameter}1"
(I also changed argument to parameter because arguments are on the caller side,
and parameters are on the function declaration side.)
================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:264
@@ -258,1 +263,3 @@
+def err_anyx86_interrupt_called : Error<
+ "interrupt service routine can't be called directly">;
def warn_mips_interrupt_attribute : Warning<
----------------
can't -> cannot
http://reviews.llvm.org/D15709
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
