[PATCH] D64242: [AArch64] Fix scalar vuqadd intrinsics operands

2019-07-08 Thread Diogo N. Sampaio via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL365300: [AArch64] Fix scalar vuqadd intrinsics operands (authored by dnsampaio, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D64242: [AArch64] Fix scalar vuqadd intrinsics operands

2019-07-05 Thread John Brawn via Phabricator via cfe-commits
john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM, with one nitpick. Comment at: test/CodeGen/aarch64-neon-vuqadd-float-conversion-warning.c:2 +// RUN: %clang_cc1 -triple arm64-none-linux-gnu -target-feature

[PATCH] D64242: [AArch64] Fix scalar vuqadd intrinsics operands

2019-07-05 Thread Diogo N. Sampaio via Phabricator via cfe-commits
dnsampaio updated this revision to Diff 208144. dnsampaio added a comment. - Fix previously existing tests Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64242/new/ https://reviews.llvm.org/D64242 Files: include/clang/Basic/arm_neon.td

[PATCH] D64242: [AArch64] Fix scalar vuqadd intrinsics operands

2019-07-05 Thread Diogo N. Sampaio via Phabricator via cfe-commits
dnsampaio created this revision. dnsampaio added a reviewer: LukeCheeseman. Herald added subscribers: cfe-commits, kristof.beyls, javed.absar. Herald added a project: clang. Change the vuqadd scalar instrinsics to have the second argument as unsigned values, not signed, accordingly to