The Zcb extension has c.lbu, but not c.lb. This patch makes us
prefer LBU over LB if we have a choice which will enable more
compression opportunities.
Details
Details
- Reviewers
reames jrtc27 asb luismarques - Commits
- rG8e43c22d3038: [RISCV] Use LBU for extloadi8.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Are there any cases like with all the *W optimisations where it would be better to sign-extend, or is i32 special there due to the *W ops and for i8 the only cases where it's beneficial will already have become sextloads?
Comment Actions
I think i32 is special. And right now we don't have any optimizations after isel that notice a difference between lbu and lb. For SExtWRemoval they are both viewed as producing a value that has more than 33 sign bits.