The IsReverseCompare flag tells CGBuiltin to swap the operands,
so that a LT/LE intrinsics can be expressed in terms of GE/GT
intrinsics.
This patch also adds builtins for the wide-variants of the compares.
Paths
| Differential D78747
[SveEmitter] Add builtins for compares and ReverseCompare flag. ClosedPublic Authored by sdesmalen on Apr 23 2020, 12:19 PM.
Details Summary The IsReverseCompare flag tells CGBuiltin to swap the operands, This patch also adds builtins for the wide-variants of the compares.
Diff Detail
Event Timeline
Comment Actions LGTM with one minor comment
This revision is now accepted and ready to land.Apr 23 2020, 12:59 PM Closed by commit rG0ddb2034c161: [SveEmitter] Add builtins for compares and ReverseCompare flag. (authored by sdesmalen). · Explain WhyApr 24 2020, 6:59 AM This revision was automatically updated to reflect the committed changes. sdesmalen marked an inline comment as done.
Revision Contents
Diff 259876 clang/include/clang/Basic/TargetBuiltins.h
clang/include/clang/Basic/arm_sve.td
clang/lib/CodeGen/CGBuiltin.cpp
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmpeq.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmpge.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmpgt.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmple.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmplt.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmpne.c
clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_cmpuo.c
clang/utils/TableGen/SveEmitter.cpp
|
Isn't "w" the same as the existing "g"? Or is there some difference I'm missing?