[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