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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
