On Tue, Nov 28, 2023 at 02:16:11PM +0100, Rainer Orth wrote:
> Hi Jakub,
>
> >>> But will they accept a patch to check a macro never set anywhere in and
> >>> irrelevant to LLVM? That's why I kept all in one patch, to be GCC-local.
> >>
> >> I meant the patch would be gcc local.
> >> But, for later we need only the changes to the imported files be in one
> >> commit, not others, because merge.sh will not revert the GCC owned changes,
> >> just the imported ones, and so that is what should be reapplied.
> >> And, the preference of not using config.h is because we do it like that
> >> for other stuff already (exactly to minimize amount of local changes).
> >
> > ah, now I get it. Will rework the patch accordingly.
>
> here are both patches.
>
> Bootstrapped on sparc-sun-solaris2.11 (as and gas) and
> i386-pc-solaris2.11 (as and gas).
>
> Ok for trunk?
Yes, thanks.
Jakub
