[PATCH] D114060: [RISCV][NFC] Simplify immediate materialisation

Ben Shi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 23:35:31 PST 2021


benshi001 added a comment.

In D114060#3136735 <https://reviews.llvm.org/D114060#3136735>, @craig.topper wrote:

> Is the compiler simplicity worth making the assembly harder to read since it won't print zext.w anymore?

The immediate materialisation already become more tricky with SHxADD/BCLRI/BSETI/SLLIUW, most assembly sequances are not obvious, so is it necessary to keep immediate materialisation readable ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114060



More information about the llvm-commits mailing list