[PATCH] D132999: [RISCV][CostModel] Add cost model for integer abs

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 15:23:13 PDT 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:480
+      return LT.first * 2;
+    }
+  }
----------------
There an implicit fallthrough here. You need a `break;`


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

https://reviews.llvm.org/D132999



More information about the llvm-commits mailing list