| Summary: | Support .llvm_addrsig section | ||
|---|---|---|---|
| Product: | gcc | Reporter: | Rui Ueyama <rui314> |
| Component: | ipa | Assignee: | Not yet assigned to anyone <unassigned> |
| Status: | SUSPENDED --- | ||
| Severity: | normal | CC: | dushistov, hubicka, ishitatsuyuki, linkw, marxin, sjames |
| Priority: | P3 | Keywords: | missed-optimization |
| Version: | 11.2.0 | ||
| Target Milestone: | --- | ||
| See Also: |
https://reviews.llvm.org/D47744 https://github.com/rui314/mold/issues/484 |
||
| Host: | Target: | ||
| Build: | Known to work: | ||
| Known to fail: | Last reconfirmed: | 2022-05-17 00:00:00 | |
|
Description
Rui Ueyama
2022-05-17 03:02:29 UTC
I see most of it was implemented in the assembler. So you might want to report the support for .addrsig directive there too. From reading the specifications of the extension, you do need assembler support first as there is no way for GCC to emit a reference to the index of the symbol table as that is not exposed via normal assembler directives. Suspended until GNU binutils support is added. I think we can implement the `.addrsig` support to the assembler, but I wonder if GCC will support it once the GNU assembler gains the feature? (In reply to Rui Ueyama from comment #3) > I think we can implement the `.addrsig` support to the assembler, but I > wonder if GCC will support it once the GNU assembler gains the feature? Yes I think it will, shouldn't be too hard to implement because IPA cgraph already has that information and uses it for ICF inside GCC. Cool! We'll add a `.llvm_addrsig` support to binutils and get back to you guys. Good, I can prepare a GCC patch once the binutils support is there. https://sourceware.org/pipermail/binutils/2022-May/121000.html I just posted an initial revision of the patchset for gas. |