Re: [PATCH] config.sub: recognise ARM64EC machine type

[Billy Laws]

On Thu, 14 Sept 2023 at 01:43, Po Lu <luangruo@yahoo.com> wrote:
>
> Billy Laws <blaws05@gmail.com> writes:
>
> > diff --git a/testsuite/config-sub.data b/testsuite/config-sub.data
> > index ba934b6..aba6ffc 100644
> > --- a/testsuite/config-sub.data
> > +++ b/testsuite/config-sub.data
> > @@ -102,6 +102,8 @@ arm64-apple-tvos                          
> > aarch64-apple-tvos
> >  arm64-apple-tvos10.0.0                               
> > aarch64-apple-tvos10.0.0
> >  arm64-apple-watchos                          aarch64-apple-watchos
> >  arm64-apple-watchos5.0                               
> > aarch64-apple-watchos5.0
> > +arm64ec-w64-mingw32                          arm64ec-w64-mingw32
>
> *-w64-* is not a system vendor, is it?  I presume this ought to be:
I just put what MinGW was using here since that is what I'm targeting, I assumed
that tests should represent what was actually being used.
>
>   arm64ec-pc-mingw64
>
> or something akin to that.
>
> > +arm64ec-windows-msvc                                 
> > arm64ec-unknown-windows-msvc
>
> *-*-windows-* are all misnomers, inasmuch as `msvc' or `gnu'
> (representing MinGW) do not constitute OS names.  The correct operating
> system for MSVC within configuration tuples is `winnt'.
LLVM doesn't even support "winnt" in triples, only "win32" and
"windows".  "-windows-msvc" is
also used throughout its testsuite.

I'll submit a v2 with the test data changed per your suggestions.