[llvm] [AArch64][GlobalISel] Take abs scalar codegen closer to SDAG (PR #84886)

David Green via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 14 00:42:25 PDT 2024


================
@@ -515,6 +515,7 @@ AArch64LegalizerInfo::AArch64LegalizerInfo(const AArch64Subtarget &ST)
   getActionDefinitionsBuilder(G_ICMP)
       .legalFor({{s32, s32},
                  {s32, s64},
+                 {s64, s64},
----------------
davemgreen wrote:

I think this and the select below is marking a s64 condition code as legal, which Im not sure is what we would want. We have the clampScalar(0, s32, s32) below.

https://github.com/llvm/llvm-project/pull/84886


More information about the llvm-commits mailing list