ABataev marked 2 inline comments as done.
================
Comment at: test/Sema/attr-x86-interrupt.c:54
@@ +53,3 @@
+ foo8((int *)argv); // expected-error {{interrupt service routine can't
be used directly}}
+ return 0;
+}
----------------
aaron.ballman wrote:
> I'd like to see a test like:
> ```
> __attribute__((interrupt)) void foo8(int *a) {}
>
> void g(void (*fp)(int *));
>
> void f() {
> g(foo8);
> }
> ```
> as I think this is supposed to diagnose. It might also be good to have a
> similar test using templates in C++.
Checked again. I did not look at the code for a long time and just forgot that
the functions cannot be called, but you can take their address. Modified the
message and the code itself for better check.
http://reviews.llvm.org/D15709
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
