[llvm] [RISCV] Add option to disable ASM compress. (PR #131670)

Francesco Petrogalli via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 18 08:18:14 PDT 2025


fpetrogalli wrote:

> > > Why do you need this as an option? There is no explanation in the PR.
> > 
> > 
> > Sure - it is needed for testing.
> 
> In LLVM, or in e.g. external projects? If only the former, we can land this, under the understanding we can remove it when exact mode lands, which I'm aiming to be before 21.0.
> 

External projects, no need to temporarily land the option.



> I wrote a proposal here, which hasn't been accepted yet but seems to have broad consensus for the main parts (disabling compression and both branch and linker relaxation) [riscv-non-isa/riscv-asm-manual#122](https://github.com/riscv-non-isa/riscv-asm-manual/pull/122)
> 

Your proposal is useful. I stated my support in there.

> And I have an in-progress PR here which I need to get back to to sort out disabling linker relaxations, hopefully this week: #122483
> 
> I believe it covers your use-cases. [...]

Indeed it does. How long will it take for the proposal to get ratified? I guess that requires more time than doing the actual implementation, right?



https://github.com/llvm/llvm-project/pull/131670


More information about the llvm-commits mailing list