Re: [PATCH v1 01/13] Introduce aarch64-w64-mingw32 target

2024-02-23 Thread Fangrui Song
+Martin who may have an opinion (https://github.com/mstorsjo/llvm-mingw supports aarch64) On Fri, Feb 23, 2024 at 6:15 AM Evgeny Karpov wrote: > > Hi Andrew and Richard, > > Thank you for pointing out there's no need for a 64-bit ISA and the > big-endian target. > These changes will be addressed

Re: [PATCH v1 01/13] Introduce aarch64-w64-mingw32 target

2024-02-22 Thread Richard Earnshaw (lists)
On 21/02/2024 18:16, Evgeny Karpov wrote: > +aarch64*-*-mingw*) Other targets are a bit inconsistent here as well, but, as Andrew mentioned, if you don't want to handle big-endian, it might be better to match aarch64-*-mingw* here. R.

Re: [PATCH v1 01/13] Introduce aarch64-w64-mingw32 target

2024-02-21 Thread Andrew Pinski
On Wed, Feb 21, 2024 at 10:17 AM Evgeny Karpov wrote: > > > need_64bit_isa=yes This is not needed in the patch as it is only used for x86_64 targets. Should you make sure nobody specifies the big-endian target: aarch64_be-w64-mingw32 ? Thanks, Andrew Pinski