[PATCH] D84287: [SelectionDAG][GISel] Make LegalizeDAG lower FNEG using integer ops.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: spatel.
RKSimon added a comment.

@arsenm Any more thoughts?



================
Comment at: llvm/lib/Target/ARM/ARMLegalizerInfo.cpp:91
       .legalFor({s32})
-      .minScalar(0, s32);
+      .clampScalar(0, s32, s32);
 
----------------
should this be a separate fix?


================
Comment at: llvm/test/CodeGen/X86/GlobalISel/legalize-fneg.mir:2
 # NOTE: Assertions have been autogenerated by utils/update_mir_test_checks.py
 # RUN: llc -O0 -run-pass=legalizer %s -o - | FileCheck %s
 
----------------
Worth adding a f80 case as well?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84287/new/

https://reviews.llvm.org/D84287