HomePhabricator

[AArch64] Fix vsqadd scalar intrinsics operands

Description

[AArch64] Fix vsqadd scalar intrinsics operands

Summary:
Change the vsqadd scalar instrinsics to have the second argument as signed values, not unsigned,
accordingly to https://developer.arm.com/architectures/instruction-sets/simd-isas/neon/intrinsics

The existing unsigned argument can cause faulty code as negative float to unsigned conversion is
undefined, which llvm/clang optimizes away.

Reviewers: LukeCheeseman, john.brawn

Reviewed By: john.brawn

Subscribers: john.brawn, javed.absar, kristof.beyls, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64239

Details

Committed
dnsampaioJul 8 2019, 1:35 AM
Reviewer
john.brawn
Differential Revision
D64239: [AArch64] Fix vsqadd scalar intrinsics operands
Parents
rL365297: [UpdateTestChecks] Skip over .Lfunc_begin for RISC-V
Branches
Unknown
Tags
Unknown