This is an archive of the discontinued LLVM Phabricator instance.

fix ARM test case - missing and incomplete RUN lines
ClosedPublic

Authored by spatel on Aug 11 2014, 10:56 AM.

Reviewers

Commits

Summary

This patch corrects a missing RUN line in the ARM codegen test for fneg ops. We should also explicitly specify +/-neonfp.

The bug was introduced at r99570 when use of "-arm-use-neon-fp" was removed.

spatel retitled this revision from to fix ARM test case - missing and incomplete RUN lines.

spatel updated this object.

spatel edited the test plan for this revision. (Show Details)

spatel added reviewers: asl, rengolin.

spatel added a subscriber: Unknown Object (MLST).

Well spotted! This looks fine to me.

Tim.

This revision is now accepted and ready to land.Aug 11 2014, 11:54 AM

Closed by commit rL215377 (authored by @spatel).

Path

Size

test/

CodeGen/

ARM/

5 lines

	; RUN: llc -mtriple=arm-eabi -mattr=+vfp2 %s -o - \			; RUN: llc -mtriple=arm-eabi -mattr=+vfp2 %s -o - \
	; RUN: \| FileCheck %s -check-prefix=VFP2			; RUN: \| FileCheck %s -check-prefix=VFP2

	; RUN: llc -mtriple=arm-eabi -mattr=+neon %s -o - \			; RUN: llc -mtriple=arm-eabi -mattr=+neon,-neonfp %s -o - \
	; RUN: \| FileCheck %s -check-prefix=NFP0			; RUN: \| FileCheck %s -check-prefix=NFP0

				; RUN: llc -mtriple=arm-eabi -mattr=+neon,+neonfp %s -o - \
				; RUN: \| FileCheck %s -check-prefix=NFP1

	; RUN: llc -mtriple=arm-eabi -mcpu=cortex-a8 %s -o - \			; RUN: llc -mtriple=arm-eabi -mcpu=cortex-a8 %s -o - \
	; RUN: \| FileCheck %s -check-prefix=CORTEXA8			; RUN: \| FileCheck %s -check-prefix=CORTEXA8