Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

NEON scalar comparison functions should return unsigned values #47687

Open Quuxplusone opened 3 years ago

Quuxplusone commented 3 years ago
Bugzilla Link PR48718
Status NEW
Importance P normal
Reported by Evan Nemerson (clang@evan.coeusgroup.com)
Reported on 2021-01-11 09:59:17 -0800
Last modified on 2021-01-12 03:00:34 -0800
Version 11.0
Hardware Other All
CC david.spickett@linaro.org, htmldeveloper@gmail.com, llvm-bugs@lists.llvm.org, smithp352@googlemail.com, Ties.Stuij@arm.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also
Some functions which compare 64-bit values return int64_t in clang instead of
uint64_t.  For example, documentation for vceqd_s64 (which returns int64_t in
clang): https://developer.arm.com/architectures/instruction-sets/simd-
isas/neon/intrinsics?search=vceqd_s64

I ran into this issue with vceqd_s64, but the other comparison functions seem
to share the same problem:

 * vceqd_s64
 * vceqzd_s64
 * vcged_s64
 * vcgezd_s64
 * vcgtd_s64
 * vcgtzd_s64
 * vcled_s64
 * vclezd_s64
 * vcltd_s64
 * vcltzd_s64
Quuxplusone commented 3 years ago

I believe I fixed these (plus vtstd_s64) in https://reviews.llvm.org/D88009. Probably didn't make it into the 11 branch.

Can you confirm if you see the mismatch on trunk still?