[PATCH] D114380: [ARM] Teach getIntImmCostInst about the cost of saturating fp converts

Wed Dec 1 06:38:16 PST 2021

dmgreen added a comment.

In D114380#3164169 <https://reviews.llvm.org/D114380#3164169>, @gbreynoo wrote:

> Hi @dmgreen, I think this change is the cause of `CodeGen/ARM/fpclamptosat.ll` failing on the `clang-x64-windows-msvc` build bot.

Yeah, I saw. I'm not really sure what's going on, I'm not sure why it being windows matters. I've tried to fix it once to no avail, and have had to get a windows build going to see what is different. I'll revert the patch in the meantime and recommit once I can figure out why it's not working as intended.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114380