dwblaikie wrote:
> This approahc parallels the strategy used for
> [-enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang](https://reviews.llvm.org/D125142),
> albeit a nicer name.
Fair enough - yeah, if we're wordsmithing this. Maybe "allow" would be a good
word, rather
smithp35 wrote:
I will take a look, but may take me a while to go through it. Some very quick
comments:
* Would `--enable-experimental-crel` be a better name than
`--experimental-crel` that makes it clear that the option is just an unlock of
another flag, and not a flag in of itself?
* I
MaskRay wrote:
Thanks for the comments.
The syntax `-Wa,--crel -Wa,--experimental-crel` (or
`-Wa,--crel,--experimental-crel`) might seem cumbersome, it serves a purpose:
* `-Wa,--experimental-crel`: no effect
* `-Wa,--crel`: driver error. Make sure users acknowledge the non-standard
status.
dwblaikie wrote:
Oh, and I take it there's no /official/ extension space in the SHT_* space? The
intent is to standardize it where it lies? (like use 20 for the shipped
version? Or eventually get some lower designation?) I understand tnhe hesitance
to use "SHT_LLVM" or the like (though
dwblaikie wrote:
Presumably this'll be split out into separate reviews for the components here?
(I'd guess llvm-mc then clang integrated assembler, with readobj and yaml2obj
in any order/don't have dependencies, unless they're needed for testing, in
which case I guess they come first?)
I
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Fangrui Song (MaskRay)
Changes
CREL is a compact relocation format for the ELF object file format.
This patch adds integrated assembler support (using the RELA form),
which can be enabled with `clang -c
llvmbot wrote:
@llvm/pr-subscribers-mc
Author: Fangrui Song (MaskRay)
Changes
CREL is a compact relocation format for the ELF object file format.
This patch adds integrated assembler support (using the RELA form),
which can be enabled with `clang -c -Wa,--crel,--experimental-crel`.
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/91280
CREL is a compact relocation format for the ELF object file format.
This patch adds integrated assembler support (using the RELA form),
which can be enabled with `clang -c -Wa,--crel,--experimental-crel`.
