diff mbox series

[committed] hppa: Revise __TIMESIZE define to use __WORDSIZE

Message ID ZC3ArGn5cOvJesRk@mx3210.localdomain
State New
Headers show
Series [committed] hppa: Revise __TIMESIZE define to use __WORDSIZE | expand

Commit Message

John David Anglin April 5, 2023, 6:40 p.m. UTC
Committed to master.

Dave
---

hppa: Revise  __TIMESIZE define to use __WORDSIZE

Handle both 32 and 64-bit ABIs.

Signed-off-by: John David Anglin <dave.anglin@bell.net>

Comments

Adhemerval Zanella Netto April 5, 2023, 7 p.m. UTC | #1
Do we support or plan to support 64 bit hppa? The patch is not wrong itself,
but I also don't see why it is required.

On 05/04/23 15:40, John David Anglin wrote:
> Committed to master.
> 
> Dave
> ---
> 
> hppa: Revise  __TIMESIZE define to use __WORDSIZE
> 
> Handle both 32 and 64-bit ABIs.
> 
> Signed-off-by: John David Anglin <dave.anglin@bell.net>
> 
> diff --git a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
> index 316a9c1ce6..cbceeb4a29 100644
> --- a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
> +++ b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
> @@ -16,5 +16,7 @@
>     License along with the GNU C Library; if not, see
>     <https://www.gnu.org/licenses/>.  */
>  
> +#include <bits/wordsize.h>
> +
>  /* Size in bits of the 'time_t' type of the default ABI.  */
> -#define __TIMESIZE	32
> +#define __TIMESIZE	__WORDSIZE
John David Anglin April 6, 2023, 9:11 p.m. UTC | #2
Hopefully.

There is 64-bit hppa support in gcc and Linux.

On 2023-04-05 3:00 p.m., Adhemerval Zanella Netto wrote:
> Do we support or plan to support 64 bit hppa? The patch is not wrong itself,
> but I also don't see why it is required.
>
> On 05/04/23 15:40, John David Anglin wrote:
>> Committed to master.
>>
>> Dave
>> ---
>>
>> hppa: Revise  __TIMESIZE define to use __WORDSIZE
>>
>> Handle both 32 and 64-bit ABIs.
>>
>> Signed-off-by: John David Anglin <dave.anglin@bell.net>
>>
>> diff --git a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
>> index 316a9c1ce6..cbceeb4a29 100644
>> --- a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
>> +++ b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
>> @@ -16,5 +16,7 @@
>>      License along with the GNU C Library; if not, see
>>      <https://www.gnu.org/licenses/>.  */
>>   
>> +#include <bits/wordsize.h>
>> +
>>   /* Size in bits of the 'time_t' type of the default ABI.  */
>> -#define __TIMESIZE	32
>> +#define __TIMESIZE	__WORDSIZE
diff mbox series

Patch

diff --git a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
index 316a9c1ce6..cbceeb4a29 100644
--- a/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
+++ b/sysdeps/unix/sysv/linux/hppa/bits/timesize.h
@@ -16,5 +16,7 @@ 
    License along with the GNU C Library; if not, see
    <https://www.gnu.org/licenses/>.  */
 
+#include <bits/wordsize.h>
+
 /* Size in bits of the 'time_t' type of the default ABI.  */
-#define __TIMESIZE	32
+#define __TIMESIZE	__WORDSIZE