Fangrui Song

Results 110 comments of Fangrui Song

Has this been merged? (not a Meta employee)

> ld maintainers do not seem to be interested. Also lld/ELF. A post-link tool implementing the feature is definitely the right direction. In a lot of cases, the user using...

I really like the proposal for improving project governance, but I also share the concerns raised about area teams in the other replies. In particular: https://github.com/llvm/llvm-www/pull/54#issuecomment-1759220410 (nikic) > ... I...

I am just trying to make `-L` in `clang -target $triple '-###'` (replace `$triple` with the directory names under `usr/lib/{gcc,gcc-cross}/`) output follow gcc more closely... And I noticed that `gcc...

Thanks for the patch. This should fix some `-fsanitize=alignment` (part of `-fsanitize=undefined`) uses.

LGTM > There is a typo in the spelling of R_

I agree that in the majority of cases linkers don't need to move a relocation from `.relr.auth.dyn` to `.rela.dyn`. The question is do we need this RELR variant? If we...

Shall we use `GDAT(S) + A` ? But that can be done after https://github.com/ARM-software/abi-aa/issues/217 has a concrete resolution.

binutils 2.26, released in Jan 2016. In 2020, I changed clang cmake option `ENABLE_X86_RELAX_RELOCATIONS` to default to on: https://github.com/llvm/llvm-project/commit/c41a18cf61790fc898dcda1055c3efbf442c14c0 Setting the option to false makes less sense after 3.5 years...