[PATCH] D54574: [SelectionDAG] Support promotion of the FPOWI integer operand

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 30 03:59:25 PST 2018


asb updated this revision to Diff 176079.
asb added a comment.

Updated to add note about FPOWI being undefined if integer operand is > 32-bits, as suggested by @efriedma

@apazos: The patch applies with minimal fuzz once you've applied the RV64F codegen match (which is marked as a dependency). I think I'd incorrectly indicated to you that you'd want to apply this patch for RV64I testing - that was incorrect. rL346034 <https://reviews.llvm.org/rL346034> is sufficient for FPOWI on RV64I. Sorry for the confusion!


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

https://reviews.llvm.org/D54574

Files:
  include/llvm/CodeGen/ISDOpcodes.h
  lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
  lib/CodeGen/SelectionDAG/LegalizeTypes.h
  test/CodeGen/RISCV/float-intrinsics.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54574.176079.patch
Type: text/x-patch
Size: 12464 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181130/06e6b2d7/attachment.bin>


More information about the llvm-commits mailing list