[Bug ada/100559] build failure of 32-bit Ada runtime after local modification

2021-05-13 Thread sumbera at volny dot cz via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100559

--- Comment #17 from Petr Sumbera  ---
(In reply to Petr Sumbera from comment #16)
> (In reply to r...@cebitec.uni-bielefeld.de from comment #15)

> > I'd seen that change before in solaris-userland, but never understood
> > the point.  Those directories are only used inside the build trees.
> 
> Ok. I will see if we can remove it for GCC 11.

I think it was done because of packaging. So that SPARC and Intel would have
more similar paths (in one manifest file). But it can be overcome. I will try
to avoid the patch file.

[Bug ada/100559] build failure of 32-bit Ada runtime after local modification

2021-05-13 Thread sumbera at volny dot cz via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100559

--- Comment #16 from Petr Sumbera  ---
(In reply to r...@cebitec.uni-bielefeld.de from comment #15)
> > --- Comment #14 from Petr Sumbera  ---
> [...]
> > It has following comment:
> >
> > #  We want sparc/i386 to match locations for their 32 bit support when 
> > building
> > #  multilib.  For now use the more generic /32.  This has not been filed
> > #  upstream.
> >
> > I wonder whether it makes sense to have it in upstream?
> 
> I'd seen that change before in solaris-userland, but never understood
> the point.  Those directories are only used inside the build trees.

Ok. I will see if we can remove it for GCC 11.

[Bug ada/100559] build failure of 32-bit Ada runtime after local modification

2021-05-13 Thread ro at CeBiTec dot Uni-Bielefeld.DE via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100559

--- Comment #15 from ro at CeBiTec dot Uni-Bielefeld.DE  ---
> --- Comment #14 from Petr Sumbera  ---
[...]
> It has following comment:
>
> #  We want sparc/i386 to match locations for their 32 bit support when 
> building
> #  multilib.  For now use the more generic /32.  This has not been filed
> #  upstream.
>
> I wonder whether it makes sense to have it in upstream?

I'd seen that change before in solaris-userland, but never understood
the point.  Those directories are only used inside the build trees.

[Bug ada/100559] build failure of 32-bit Ada runtime after local modification

2021-05-13 Thread sumbera at volny dot cz via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100559

--- Comment #14 from Petr Sumbera  ---
(In reply to Eric Botcazou from comment #13)
> > $ cat gcc-11.1.0/gcc/config/sparc/t-sol2
> > MULTILIB_OPTIONS = m32/m64
> > MULTILIB_DIRNAMES = 32 sparcv9
> > MULTILIB_MATCHES =
> > MULTILIB_OSDIRNAMES = . sparcv9
> 
> So you have a local modification after all, here is the pristine file:
> 
> eric@fomalhaut:~/tmp/gcc-11.1.0> cat gcc/config/sparc/t-sol2
> MULTILIB_OPTIONS = m32/m64
> MULTILIB_DIRNAMES = sparcv8plus sparcv9
> MULTILIB_MATCHES =
> MULTILIB_OSDIRNAMES = . sparcv9
> 
> You need to do s/sparcv8plus/32/ on $(srcdir)/gcc/ada/Makefile.rtl as well.

Thank you very much! I'm trying it now.

I don't know the history of the change:

https://github.com/oracle/solaris-userland/blob/master/components/gcc10/patches/001-multilib-sparc.patch

It has following comment:

#  We want sparc/i386 to match locations for their 32 bit support when building
#  multilib.  For now use the more generic /32.  This has not been filed
#  upstream.

I wonder whether it makes sense to have it in upstream?

[Bug ada/100559] build failure of 32-bit Ada runtime after local modification

2021-05-13 Thread ebotcazou at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100559

Eric Botcazou  changed:

   What|Removed |Added

  Build||sparcv9-sun-solaris2.*
 Target||sparcv9-sun-solaris2.*
 Resolution|--- |INVALID
 Status|WAITING |RESOLVED
   Host||sparcv9-sun-solaris2.*
Summary|Solaris SPARC GCC 11.1 Ada  |build failure of 32-bit Ada
   |build: i-cexten.ads:278:28: |runtime after local
   |modulus exceeds limit (2 ** |modification
   |64) |

--- Comment #13 from Eric Botcazou  ---
> $ cat gcc-11.1.0/gcc/config/sparc/t-sol2
> MULTILIB_OPTIONS = m32/m64
> MULTILIB_DIRNAMES = 32 sparcv9
> MULTILIB_MATCHES =
> MULTILIB_OSDIRNAMES = . sparcv9

So you have a local modification after all, here is the pristine file:

eric@fomalhaut:~/tmp/gcc-11.1.0> cat gcc/config/sparc/t-sol2
MULTILIB_OPTIONS = m32/m64
MULTILIB_DIRNAMES = sparcv8plus sparcv9
MULTILIB_MATCHES =
MULTILIB_OSDIRNAMES = . sparcv9

You need to do s/sparcv8plus/32/ on $(srcdir)/gcc/ada/Makefile.rtl as well.