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?