Re: [PATCH] [Ada] Make clockid_t type public on GNU/kFreeBSD

2019-07-04 Thread James Clarke
I don't have commit access so could you please do so on my behalf? This bug applies to the 9 branch too, so please consider backporting it. Thanks, James > On 4 Jul 2019, at 07:50, Arnaud Charlet wrote: > > OK, thanks. > >> From: James Clarke >> >> Monot

[PATCH] [Ada] Make clockid_t type public on GNU/kFreeBSD

2019-07-03 Thread James Clarke
From: James Clarke Monotonic_Clock and RT_Resolution in the recently-added s-tpopmo.adb call clock_gettime/clock_getres with the integral constants from OSC and thus rely on clockid_t being an integral type, so we cannot hide it on GNU/kFreeBSD. Instead, make the definition public to match all

[PATCH] Put __dso_handle in .sdata/.sbss on ia64

2019-06-01 Thread James Clarke
The symbol is exposed to C by dso_handle.h, and since it's a single 8-byte pointer, it is just within the threshold for being in the small data (or bss) section, so code accessing it will use GP-relative addressing. Therefore we must put it in .sdata/.sbss in case our other data sections grow too b

Re: [PATCH] PR bootstrap/87338: Fix ia64 bootstrap comparison regression in r257511

2019-05-21 Thread James Clarke
On 26 Apr 2019, at 15:24, Jeff Law wrote: > On 4/26/19 2:01 AM, Jakub Jelinek wrote: >> On Fri, Apr 26, 2019 at 08:58:18AM +0200, Richard Biener wrote: >>> On Thu, Apr 25, 2019 at 7:52 PM James Clarke wrote: >>>> >>>> By using ASM_OUTPUT_LABEL, r

[PATCH] PR bootstrap/87338: Fix ia64 bootstrap comparison regression in r257511

2019-04-25 Thread James Clarke
By using ASM_OUTPUT_LABEL, r257511 forced the assembler to start a new bundle when emitting an inline entry label on. Instead, use ASM_OUTPUT_DEBUG_LABEL like for the block begin and end labels so tags are emitted rather than labels. gcc/ PR bootstrap/87338 * dwarf2out.c (dwarf2out

[PATCH] RISC-V: Pass -mno-relax through to assembler if supported

2018-11-18 Thread James Clarke
GCC will emit ".option (no)relax" in its outputted assembly, but when using it as an assembler driver, such as for preprocessed assembly, it's merely preprocessing and therefore will not generate this directive. Therefore we should pass -mno-relax on to the assembler if specified (and supported) as

[PATCH v2] Enable decimal float on x86_64 kFreeBSD and Hurd

2018-07-04 Thread James Clarke
config/ * dfp.m4 (enable_decimal_float): Enable for x86_64*-*-gnu* to catch x86_64 kFreeBSD and Hurd. gcc/ * configure: Regenerate. libdecnumber/ * configure: Regenerate. libgcc/ * configure: Regenerate. --- Hi, Apologies; I accidentally sent an older ver

[PATCH] Enable decimal float on x86_64 kFreeBSD and Hurd

2018-07-04 Thread James Clarke
config/ * dfp.m4 (enable_decimal_float): Enable for x86_64*-*-gnu* to catch x86_64 kFreeBSD and Hurd. gcc/ * configure: Regenerate. libdecnumber/ * configure: Regenerate. libgcc/ * configure: Regenerate. --- config/dfp.m4 | 2 +- gcc/configure

[PATCH v2] PR rtl-optimization/83565: Fix 32-bit rotate on ia64

2017-12-23 Thread James Clarke
On ia64, 32-bit rotates are implemented by copying the lower 32 bits of a register into the upper half, then performing a right shift. However, depending on the bit pattern in question, this can leave the upper 32 bits as non-zero, despite being only a 32-bit unsigned result. Therefore add an extra

[PATCH] PR rtl-optimization/83565: Fix 32-bit rotate on ia64

2017-12-23 Thread James Clarke
From: James Clarke On ia64, 32-bit rotates are implemented by copying the lower 32 bits of a register into the upper half, then performing a right shift. However, depending on the bit pattern in question, this can leave the upper 32 bits as non-zero, despite being only a 32-bit unsigned result

[PATCH v2] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-10 Thread James Clarke
libsanitizer: PR sanitizer/78992 * sanitizer_common/sanitizer_platform_limits_posix.h (struct __sanitizer_sigaction): Cherry-pick upstream r291561. --- libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.h | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(

[PATCH] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-04 Thread James Clarke
libsanitizer: PR sanitizer/78992 * sanitizer_common/sanitizer_platform_limits_posix.h: sigaction should only have __glibc_reserved0 as a member on 64-bit sparc. --- libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.h | 7 +-- 1 file changed, 5 insertions(+),

[PATCH] Fix ICE for sparc targets in function_arg_record_value (PR target/77759)

2016-10-05 Thread James Clarke
gcc/ PR target/77759 * config/sparc/sparc.c (classify_data_t): Remove unused int_regs field. (classify_registers): Don't set int_regs. (function_arg_slotno): Don't initialise int_regs. Check slotno is within range for empty structs, just like int register-onl

Re: [PATCH v2] Ensure source_date_epoch is always initialised

2016-05-24 Thread James Clarke
Hi, > On 24 May 2016, at 11:59, Dhole wrote: > > Hey! > > I'm the original author of the SOURCE_DATE_EPOCH patch. > > I've just seen this. I believe that this bug was fixed in the the > rework of the patch I sent some days ago [1], although the latest > version of that patch has not been revie

[PATCH v2] Ensure source_date_epoch is always initialised

2016-05-19 Thread James Clarke
gcc/c-family PR preprocessor/71183 * c-common.c (get_source_date_epoch): Move to libcpp/init.c. * c-common.h (get_source_date_epoch): Remove definition, as it is now internal to libcpp/init.c. * c-lex.c (c_lex_with_flags): Remove source_date_epoch ini

[PATCH] Ensure source_date_epoch is always initialised

2016-05-18 Thread James Clarke
gcc/c-family PR preprocessor/71183 * c-common.c (get_source_date_epoch): Move to libcpp/init.c. * c-common.h (get_source_date_epoch): Remove definition, as it is now internal to libcpp/init.c. * c-lex.c (c_lex_with_flags): Remove source_date_epoch ini

Re: [PATCH v4 0/2] Fix OS X Yosemite-related issues

2014-09-06 Thread James Clarke
Just checking on the status of this as I haven't heard anything further for a while. James Clarke > On 26 Aug 2014, at 08:11, James Clarke wrote: > > These patches combine to address PR target/61407, meaning that GCC is > able to bootstrap on OS X Yosemite. > > Change

[PATCH v4 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The _DARWIN_FEATURE_64_BIT_INODE macro indicates whether dirent.d_ino is 64-bit or not, which should be sized appropriately, and whether dirent.d_seekoff should be defined/checked. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_limits_posix.cc: Only check dirent.d_seekoff o

[PATCH v4 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v4 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
_DARWIN_FEATURE_64_BIT_INODE macro is defined rather than its value James Clarke (2): Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ Fixed dirent sanitizer for 32-bit OS X builds gcc/config/darwin-c.c | 25 -- gcc/config/darwin-driver.c

[PATCH v3 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The _DARWIN_FEATURE_64_BIT_INODE macro indicates whether dirent.d_ino is 64-bit or not, which should be sized appropriately, and whether dirent.d_seekoff should be defined/checked. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_limits_posix.cc: Only check dirent.d_seekoff o

[PATCH v3 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v3 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
public _DARWIN_FEATURE_64_BIT_INODE macro rather than the internal __DARWIN_64_BIT_INO_T macro James Clarke (2): Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ Fixed dirent sanitizer for 32-bit OS X builds gcc/config/darwin-c.c | 25

[PATCH v2 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-26 Thread James Clarke
The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino is 64-bit or not, and so should be sized appropriately. Additionally, dirent.d_seekoff should only be defined/checked when __DARWIN_64_BIT_INO_T is defined. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_l

[PATCH v2 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-26 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch

[PATCH v2 0/2] Fix OS X Yosemite-related issues

2014-08-26 Thread James Clarke
These patches combine to address PR target/61407, meaning that GCC is able to bootstrap on OS X 10.10 Yosemite. James Clarke (2): Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ Fixed dirent sanitizer for 32-bit OS X builds gcc/config/darwin-c.c | 25

Re: [PATCH 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-25 Thread James Clarke
So I should just take that particular check out then? James Clarke > On 25 Aug 2014, at 21:05, Mike Stump wrote: > >> On Aug 25, 2014, at 11:03 AM, James Clarke wrote: >> Previously, this macro had taken the form 10X0, where X is the minor >> version number, e.g. 1090

Re: [PATCH 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-25 Thread James Clarke
Please note that this fixes PR 61407 (forgot to add it to libsanitizer/ChangeLog). James Clarke > On 25 Aug 2014, at 19:03, James Clarke wrote: > > The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino > is 64-bit or not, and so should be sized appropriately.

[PATCH 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-25 Thread James Clarke
The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino is 64-bit or not, and so should be sized appropriately. Additionally, dirent.d_seekoff should only be defined/checked when __DARWIN_64_BIT_INO_T is defined. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_l

[PATCH 1/2] Fix __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__

2014-08-25 Thread James Clarke
Previously, this macro had taken the form 10X0, where X is the minor version number, e.g. 1090 for OS X 10.9 Mavericks. However, as of OS X 10.10 Yosemite, it should take the form 10XX00, i.e. 101000 for 10.10. Added a test for the new format and fixed the formatting for the existing ones. gcc/Ch