[PATCH] D28451: [AVR] Add support for the 'interrupt' and 'naked' attributes

Dylan McKay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 5 16:42:58 PST 2017


dylanmckay marked 4 inline comments as done.
dylanmckay added inline comments.


================
Comment at: test/CodeGen/avr/attributes/interrupt.m:4
+// CHECK: define void @_Z3foov() #0
+void foo() __attribute__((interrupt)) { }
+
----------------
aaron.ballman wrote:
> This is not an Objective-C method decl, so it doesn't really test anything new. The test I was envisioning was something like:
> ```
> @interface F // There's probably an expected warning here about a missing base class
> -(void) foo __attribute__((interrupt));
> @end
> 
> @implementation F
> -(void) foo __attribute__((interrupt)) {}
> @end
> ```
Removed Obj-C method support.


https://reviews.llvm.org/D28451





More information about the cfe-commits mailing list