[PATCH] D63433: [RISCV] Add RISCV-specific TargetTransformInfo

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 09:12:55 PDT 2019


lenary updated this revision to Diff 205835.
lenary added a comment.

Address review feedback:

- Return `TCC_Free` in many more unknown cases, to prevent constant hoisting
- Only return `TCC_Free` for 12-bit immediates in arithmetic operations
- Use RISCVMatInt::getIntMatCost to accurately estimate materialisation cost
- Remove some useless lines and update comments and asserts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63433

Files:
  llvm/lib/Target/RISCV/CMakeLists.txt
  llvm/lib/Target/RISCV/LLVMBuild.txt
  llvm/lib/Target/RISCV/RISCVTargetMachine.cpp
  llvm/lib/Target/RISCV/RISCVTargetMachine.h
  llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
  llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h
  llvm/test/CodeGen/RISCV/imm-cse.ll
  llvm/test/Transforms/ConstantHoisting/RISCV/immediates.ll
  llvm/test/Transforms/ConstantHoisting/RISCV/lit.local.cfg

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63433.205835.patch
Type: text/x-patch
Size: 9667 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190620/2e90ffc8/attachment.bin>


More information about the llvm-commits mailing list