[PATCH] D111433: [PowerPC] Respect rounding mode in the back end
Nemanja Ivanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 19 02:28:27 PDT 2021
nemanjai added inline comments.
================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:5228
+ RetOpc = PPCISD::CALL;
+ if (IsStrictFPCall) {
+ switch (RetOpc) {
----------------
nemanjai wrote:
> qiucf wrote:
> > Better to move this into a static function?
> I don't have a particularly strong opinion on this since we are already in a static function and the code would read exactly the same. I already separated it into code that computes the node to use and then code that updates the opcode for strictfp, so it can very easily be two separate functions.
>
> If you have a preference for moving it out, please let me know and I'll update it.
@qiucf I am waiting to hear from you whether you insist on this change or if you're OK with my rationale above.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111433/new/
https://reviews.llvm.org/D111433
More information about the llvm-commits
mailing list