[PATCH] D69870: [builtins] Implement rounding mode support for i386/x86_64

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 10:18:27 PST 2019


craig.topper added inline comments.


================
Comment at: compiler-rt/lib/builtins/i386/fp_mode.c:43-45
+  cw &= X87_RMODE_MASK;
+
+  switch (cw) {
----------------
nickdesaulniers wrote:
> `switch (cw & X87_RMODE_MASK)`. LLVM will optimize such expressions in a switch.
By the time its gets to IR and goes through mem2reg/SROA it should look the same right?


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D69870





More information about the llvm-commits mailing list