[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