[llvm] [RISCV] Place mergeable small read only data into srodata section (PR #82214)
Kito Cheng via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 19 17:53:14 PST 2024
kito-cheng wrote:
> I am still interested in changing the default limit from 8 to 0. Is there interest for GCC to do so?
>From the benchmarking aspect, it's hard to say I am interesting on change the default value honestly, since I didn't got the point about the disadvantage when the GP relaxation is turn off, but it did help when GP relaxation turn on, however I may missed something, so I am happy to discuss this more, and change my mind IF we gather enough arguments :)
| | w/ GP relax | w/o GP relax |
| ---------------------- | ----------------------------------- | ------------------- |
| Default limit 0 | Less chance on GP relax `*1` | No difference?`*3` |
| Default limit 8 | More chance on GP relax `*2` | No difference |
`*1`: because the range is limited, so same range cover more symbol have a higher winning rate.
`*2`: Compare to `Default limit 0`
`*3`: I guess I may missed something here, so need your input here.
https://github.com/llvm/llvm-project/pull/82214
More information about the llvm-commits
mailing list