[PATCH] D105957: [PowerPC] Implement intrinsics for mtfsf[i]

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsPowerPC.td:1589
                    [IntrNoMem, IntrHasSideEffects]>;
   def int_ppc_mtfsfi
       : GCCBuiltin<"__builtin_ppc_mtfsfi">,
----------------
qiucf wrote:
> If `mtfsf` was changed, `mtfsfi` `mtfsb0` `mtfsb1` should also be changed?
There is no need for custom code generation for those. The reason we had to change `mtfsf` is because we need to take an integer and convert it to a double (and not a bitcast, but a `uinttofp` which I found a bit surprising).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105957