[PATCH] Implement getNoopForMachoTarget for SPARC

[Brad Smith]

On Mon, Sep 01, 2014 at 11:52:34AM -0700, Venkatraman Govindaraju wrote:
> LGTM. Please commit.
> 
> One minor comment for the testcase: Why FP suffix for the
> -check-prefix in the test case? Also, Use CHECK-FP-LABEL instead of
> CHECK-FP to match the function name in the test case for better error
> messages.
> 
> Thanks,
> Venkat
 
Like so?


Index: empty-functions.ll
===================================================================
--- empty-functions.ll	(revision 0)
+++ empty-functions.ll	(working copy)
@@ -0,0 +1,8 @@
+; RUN: llc < %s -mtriple=sparc-unknown-openbsd -disable-fp-elim | FileCheck -check-prefix=CHECK-FP-LABEL %s
+
+define void @func() {
+entry:
+  unreachable
+}
+; CHECK-FP-LABEL:      {{_?}}func:
+; CHECK-FP-LABEL: nop {{[;!]}} avoids zero-length function

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.