[PATCH] D102918: [RISCV] Avoid undef result due to promotion of FPOWI exponent

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 21 08:04:54 PDT 2021


craig.topper added a comment.

Where is it documented that larger than i32 is undefined? I see this in ISDOpcodes.def

For FPOWI, the result is undefined if if the integer operand doesn't fit into 32 bits.

That sounds more like it’s talking about the range of the value than the type. So why isn’t it ok to sign extend  i32 to i64?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102918



More information about the llvm-commits mailing list