diff mbox

Please revert the patches in bug #54040 and #59346 and special case x32

Message ID 2152307.k9oI5otnUO@polaris
State New
Headers show

Commit Message

Eric Botcazou April 9, 2014, 9:59 p.m. UTC 
> Now I got
> 
> /export/build/gnu/gcc-x32/build-x86_64-linux/./gcc/xgcc
> -B/export/build/gnu/gcc-x32/build-x86_64-linux/./gcc/
> -B/usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/bin/
> -B/usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/lib/ -isystem
> /usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/include -isystem
> /usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/sys-include    -c -g -O2
> -mx32 -fpic  -W -Wall -gnatpg -nostdinc -mx32  s-osinte.adb -o
> s-osinte.o
> 
> s-osinte.adb:101:17: operator for type "System.Linux.time_t" is not
> directly visible
> s-osinte.adb:101:17: use clause would make operation legal
> make[11]: *** [s-osinte.o] Error 1

Probably:

Comments

H.J. Lu April 9, 2014, 10:05 p.m. UTC | #1 
On Wed, Apr 9, 2014 at 2:59 PM, Eric Botcazou <ebotcazou@adacore.com> wrote:
>> Now I got
>>
>> /export/build/gnu/gcc-x32/build-x86_64-linux/./gcc/xgcc
>> -B/export/build/gnu/gcc-x32/build-x86_64-linux/./gcc/
>> -B/usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/bin/
>> -B/usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/lib/ -isystem
>> /usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/include -isystem
>> /usr/gcc-4.9.0-x32/x86_64-unknown-linux-gnu/sys-include    -c -g -O2
>> -mx32 -fpic  -W -Wall -gnatpg -nostdinc -mx32  s-osinte.adb -o
>> s-osinte.o
>>
>> s-osinte.adb:101:17: operator for type "System.Linux.time_t" is not
>> directly visible
>> s-osinte.adb:101:17: use clause would make operation legal
>> make[11]: *** [s-osinte.o] Error 1
>
> Probably:
>
> Index: s-osinte-x32.adb
> ===================================================================
> --- s-osinte-x32.adb    (revision 209244)
> +++ s-osinte-x32.adb    (working copy)
> @@ -90,6 +90,7 @@ package body System.OS_Interface is
>        S : time_t;
>        F : Duration;
>
> +      use type System.Linux.time_t;
>     begin
>        S := time_t (Long_Long_Integer (D));
>        F := D - Duration (S);
>

It compiles.  I will run GCC test.

Thanks.
Eric Botcazou April 9, 2014, 11:14 p.m. UTC | #2 
> It compiles.  I will run GCC test.

Thanks.  I installed the fixlets in the meantime.
diff mbox

Patch

Index: s-osinte-x32.adb
===================================================================
--- s-osinte-x32.adb    (revision 209244)
+++ s-osinte-x32.adb    (working copy)
@@ -90,6 +90,7 @@  package body System.OS_Interface is
       S : time_t;
       F : Duration;
 
+      use type System.Linux.time_t;
    begin
       S := time_t (Long_Long_Integer (D));
       F := D - Duration (S);