"duanbo (C)" writes:
> Hi
>
> Thank you for your suggestions.
> I have adjust the testcases and used this command to test .
> "runtest --tool gcc --tool_opts -mabi=ilp32 aarch64.exp=
> reload-valid-spoff.c"
>
> The results of those tests changed from unexpected failures to
> unsupported.
T
mailto:christophe.l...@linaro.org]
> > Sent: Wednesday, April 22, 2020 2:41 PM
> > To: duanbo (C) ; GCC Patches > patc...@gcc.gnu.org>; Richard Sandiford
> > Subject: Re: [PATCH] aarch64:Add an error message in large code model for
> > ilp32 [PR94577]
> >
gt; >> >> >
> > >> >> > It would be good to also update the documentation for
> > >> >> > -mcmodel=large to
> > >> >> state it is incompatible with -fpic, -fPIC and -mabi=ilp32.
> > >> >> >
> > >>
C)" writes:
> > Hi
> >
> >> -Original Message-
> >> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> >> Sent: Monday, April 20, 2020 10:42 PM
> >> To: duanbo (C)
> >> Cc: GCC Patches
> >> Subject: Re
"duanbo (C)" writes:
> Hi
>
>> -Original Message-
>> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
>> Sent: Monday, April 20, 2020 10:42 PM
>> To: duanbo (C)
>> Cc: GCC Patches
>> Subject: Re: [PATCH] aarch64:Add an err
Hi
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Monday, April 20, 2020 10:42 PM
> To: duanbo (C)
> Cc: GCC Patches
> Subject: Re: [PATCH] aarch64:Add an error message in large code model for
> ilp32 [PR94577]
>
s.
>> >
>> > A few minor nitpics:
>> >
>> > + PR target/94577
>> > + * gcc.target/aarch64/pr94577.c : New test
>> >
>> > Just like comments, there should be a '.' at the end of changelog entries.
>> >
>>
gt; >
> > AFAICT the format isn't exactly specified, but the email header should be
> like:
> >
> > [PATCH][AArch64] PR94577: Add an error message in large code model for
> > ilp32
> >
> > Sometimes the PR number is also placed in brackets.
> >