[PATCH] D52962: [RISCV] Constant materialisation for RV64I

[Alex Bradbury via Phabricator via llvm-commits]

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

Updated patch to add a TODO to indicate that it may sometimes be preferable to load from the constant pool. As Bruce points out, this isn't always a clear win.

If people are happy with this approach, my intent with this patchset is to land something that generates "reasonable" code and leave tuning such as constpool vs materialisation for future patches.


https://reviews.llvm.org/D52962

Files:
  lib/Target/RISCV/RISCVISelDAGToDAG.cpp
  test/CodeGen/RISCV/imm.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52962.169200.patch
Type: text/x-patch
Size: 8834 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181011/960bf74e/attachment.bin>