[PATCH] Implement getNoopForMachoTarget for SPARC

Brad Smith brad at comstyle.com
Mon Sep 1 18:09:41 PDT 2014


On 01/09/14 2:52 PM, 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.

I just copied the test from one of the other archs empty-functions.ll
test. Can't remember which arch. Probably PowerPC.

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




More information about the llvm-commits mailing list