[PATCH] D140878: [PowerPC][GISel]fcmp support

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:3314
 def : Pat<(i64 (anyext i1:$in)),
-          (SELECT_I8 $in, (LI8 1), (LI8 0))>;
+          (SELECT_I8 i1:$in, (LI8 1), (LI8 0))>;
 
----------------
Some workarounds for GlobalISelEmitter to handle this anyext pattern. This looks like an issue in `GlobalISelEmitter`, otherwise there will be many same manual modifications.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:3720
   defm : CRNotPat<(i1 (SetCC Ty:$s1, Ty:$s2, SETUGE)),
-                  (EXTRACT_SUBREG (FCmp $s1, $s2), sub_lt)>;
+                  (EXTRACT_SUBREG (FCmp Ty:$s1, Ty:$s2), sub_lt)>;
   defm : CRNotPat<(i1 (SetCC Ty:$s1, Ty:$s2, SETGE)),
----------------
Some workarounds for GlobalISelEmitter to handle this `setcc` pattern


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140878