This is an archive of the discontinued LLVM Phabricator instance.

[Builtin][ARM] Fix subsf3vfp for non-HF target
ClosedPublic

Authored by weimingz on Feb 11 2017, 8:49 PM.

Reviewers

rengolin
compnerd

Commits

*sigh* Thanks for fixing this. Please ask @hans to cherry-pick this to 4.0 as well.

This revision is now accepted and ready to land.Feb 11 2017, 8:53 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

compiler-rt/

trunk/

lib/

builtins/

arm/

2 lines

	Show All 15 Lines
	// using the Darwin calling convention where single arguments are passsed			// using the Darwin calling convention where single arguments are passsed
	// like 32-bit ints.			// like 32-bit ints.
	//			//
	.syntax unified			.syntax unified
	.p2align 2			.p2align 2
	DEFINE_COMPILERRT_FUNCTION(__subsf3vfp)			DEFINE_COMPILERRT_FUNCTION(__subsf3vfp)
	#if defined(COMPILER_RT_ARMHF_TARGET)			#if defined(COMPILER_RT_ARMHF_TARGET)
	vsub.f32 s0, s0, s1			vsub.f32 s0, s0, s1
	#elsee			#else
	vmov s14, r0 // move first param from r0 into float register			vmov s14, r0 // move first param from r0 into float register
	vmov s15, r1 // move second param from r1 into float register			vmov s15, r1 // move second param from r1 into float register
	vsub.f32 s14, s14, s15			vsub.f32 s14, s14, s15
	vmov r0, s14 // move result back to r0			vmov r0, s14 // move result back to r0
	#endif			#endif
	bx lr			bx lr
	END_COMPILERRT_FUNCTION(__subsf3vfp)			END_COMPILERRT_FUNCTION(__subsf3vfp)

	NO_EXEC_STACK_DIRECTIVE			NO_EXEC_STACK_DIRECTIVE