[PATCH v1 7/9] aarch64: Disable the anchors

2024-09-04 Thread Evgeny Karpov
Monday, September 2, 2024 Andrew Pinski wrote: > Could you expand on this and why you think disabling is correct? > It is so you could do: >         adrp    x0, .LANCHOR0 >         add     x2, x0, :lo12:.LANCHOR0 >         ldr     w1, [x0, #:lo12:.LANCHOR0] >         ldr     w0, [x2, 4] > > Rathe

Re: [PATCH v1 7/9] aarch64: Disable the anchors

2024-09-02 Thread Andrew Pinski
On Mon, Sep 2, 2024 at 6:12 AM Evgeny Karpov wrote: > > The anchors have been disabled as they use symbol + offset, which is > not applicable for COFF AArch64. This does not make sense to me at all. Anchors are a small optimization to group together some static decls so that you could reuse an an

[PATCH v1 7/9] aarch64: Disable the anchors

2024-09-02 Thread Evgeny Karpov
The anchors have been disabled as they use symbol + offset, which is not applicable for COFF AArch64. gcc/ChangeLog: * config/aarch64/aarch64.cc (TARGET_MIN_ANCHOR_OFFSET): Keep default TARGET_MAX_ANCHOR_OFFSET for PECOFF target. (TARGET_MAX_ANCHOR_OFFSET): Likewise. ---