Evgeny Karpov <evgeny.kar...@microsoft.com> writes:
> Hello,
>
> Thank you for reviewing v2!
> v3 addresses all comments on v2.
>
> Changes in v3:
> - Refactor implementation for the offset limit extension in
>   "symbol + offset" from 1MB to 16MB.
> - Apply HOST_WIDE_INT_PRINT_UNSIGNED in ASM_OUTPUT_LOCAL.
> - Exclude the "Disable the anchors" patch as it is not needed after
>   the offset limit refactoring.
> - Exclude the "Handle alignment when it is bigger than BIGGEST_ALIGNMENT"
>   patch as it is not needed anymore when FFmpeg is compiled without LTO.
>   Also, the issue could be addressed in the FFmpeg package when LTO
>   is needed.
> - Bypass hidden attribute warnings in MinGW until it is implemented.

Thanks for the update.  As usual, I can only comment on the GCC internals,
without much knowledge of how mingw actually behaves.  On that basis,
I've replied to two of the patches individually, but the series otherwise
looks good to me.  OK for trunk with those changes if there are no objections
by noon UTC on Thursday.

I can't remember if I've asked this before, but would you like commit
access?  If so, please follow the process on https://gcc.gnu.org/gitwrite.html
listing me as sponsor.

Richard

> Regards,
> Evgeny
>
>
> Evgeny Karpov (8):
>   Support weak references
>   aarch64: Add debugging information
>   aarch64: Add minimal C++ support
>   aarch64: Exclude symbols using GOT from code models
>   aarch64: Multiple adjustments to support the SMALL code model
>     correctly
>   aarch64: Extend the offset limit in "symbol + offset" from 1MB to 16MB
>   Add LTO support
>   aarch64: Bypass hidden attribute warnings in MinGW
>
>  gcc/config.gcc                    |  1 +
>  gcc/config/aarch64/aarch64-coff.h | 22 +++-------
>  gcc/config/aarch64/aarch64.cc     | 34 ++++++++++++++-
>  gcc/config/aarch64/cygming.h      | 69 +++++++++++++++++++++++++++++--
>  gcc/config/i386/cygming.h         | 16 +++----
>  gcc/config/i386/i386-protos.h     |  2 -
>  gcc/config/mingw/winnt-dll.cc     |  4 +-
>  gcc/config/mingw/winnt.cc         | 33 ++++++++++-----
>  gcc/config/mingw/winnt.h          |  7 ++--
>  libgcc/config.host                |  1 +
>  libgcc/config/aarch64/t-mingw     |  2 +
>  libiberty/simple-object-coff.c    |  4 +-
>  12 files changed, 146 insertions(+), 49 deletions(-)
>  create mode 100644 libgcc/config/aarch64/t-mingw

Reply via email to