On Fri, Jan 13, 2023 at 5:56 PM liuhongt wrote:
>
> Patches [1] and [2] fixed PR55522 for x86-linux but left all other x86
> targets unfixed (x86-cygwin, x86-darwin and x86-mingw32).
> This patch applies a similar change to other specs using crtfastmath.o.
>
> Ok for trunk?
>
The patches in [1],[2
Register-register move instructions that can be easily seen as
unnecessary by the human eye may remain in the compiled result.
For example:
/* example */
double test(double a, double b) {
return __builtin_copysign(a, b);
}
test:
add.n a3, a3, a3
extui a5, a5, 31, 1
s
In the case of the CALL0 ABI, values that must be retained before and
after function calls are placed in the callee-saved registers (A12
through A15) and referenced later. However, it is often the case that
the save and the reference are each only once and a simple register-
register move (the fra
Hi Segher,
on 2023/1/16 23:24, Segher Boessenkool wrote:
> On Mon, Jan 16, 2023 at 09:05:38PM +0800, Kewen.Lin wrote:
>>> The *_ok things should only be used for features that can be disabled
>>> during configuration, or features that we *want* users to be able to
>>> turn off (like FP, VMX, VSX,
Use the GNATMAKE variables consistently.
Avoids failures when bootstraping with a custom GNATMAKE value.
gcc/ada/ChangeLog:
* Make-generated.in: Use GNATMAKE.
* gcc-interface/Makefile.in: Ditto.
Signed-off-by: Peter Foley
---
gcc/ada/Make-generated.in | 6 +++---
gcc/ad
On 1/16/23 18:12, Peter Foley wrote:
There's no reason to create this file if none of the serial configure
options are passed.
v2: Use test instead of [ to avoid running afoul of autoconf quoting.
ChangeLog:
* configure: Regenerate.
* configure.ac: Only create serdep.tmp if
There's no reason to create this file if none of the serial configure
options are passed.
v2: Use test instead of [ to avoid running afoul of autoconf quoting.
ChangeLog:
* configure: Regenerate.
* configure.ac: Only create serdep.tmp if needed
Signed-off-by: Peter Foley
---
c
Ping?
在 2023/1/13 上午11:27, Lulu Cheng 写道:
Co-authored-by: Yang Yujie
gcc/ChangeLog:
* config/loongarch/loongarch.cc (loongarch_classify_address):
Add precessint for CONST_INT.
(loongarch_print_operand_reloc): Operand modifier 'c' is supported.
(loongarch_print_
Since r5-172-gd9f069ab4f6450, the code no longer matches the
comment as the code for Solaris 9 support was removed.
This just updates the comment to reference AIX only as
the code does.
Committed as obvious.
gcc/testsuite/ChangeLog:
* lib/target-supports.exp (add_options_for_tls): Remove
On Fri, 13 Aug 2021, Tobias Burnus wrote:
> Update the OpenMP section again, now that Jakub has added the 'masked'
> construct.
>
> Comments?
Just a little follow-up change that I pushed; see below.
Ger "blast from the past" ald
commit dcb59089cf4bd7f09f273d7b3f35c568b1d407fb
Author: Gerald Pf
Hi Tobias,
On Mon, 16 Jan 2023, Tobias Burnus wrote:
>> requires_offload, unified_address
>> - and unified_shared_memory clauses cause that the
>> - only available device is the initial device (the host). Fortran now
>> + and unified_shared_memory clauses imply the initial
>
Source: gcc-snapshot
Version: 20230108-1
Severity: important
Tags: patch
User: debian-h...@lists.debian.org
Usertags: hurd
X-Debbugs-CC: debian-h...@lists.debian.org
Hi,
gcc-snapshot in sid FTBFS on hurd-i386 due to that some patches are not
applied when building gcc-snapshot. After the statement
Dear all,
it appears that the fix for pr107874 uncovered a latent bug
for the case of arrays of type character and size zero when
passed to the intrinsics MERGE and SPREAD as SOURCE. In that
case, there is no constructor from which we could obtain
another character length. A reasonable solution
Dear all,
here's an obvious patch for an ICE-on-invalid where we missed
a check that we actually have an integer length before trying
to extract a usable value.
Regtested on x86_64-pc-linux-gnu.
I intend to commit it to mainline within the next 24h unless
there are comments.
Cheers,
Harald
dif
On Thu, 4 Apr 2019, Paul Hua wrote:
> This is a updated version, Ok for commit?
Yikes - yes! Apologies, I somehow missed that mail back then (in the
middle of two weeks essentially off the grid).
I pushed your patch with some minor changes; see below.
Gerald
commit bfea0a7904c170d262ca4b18ae
On Mon, Jan 16, 2023 at 7:56 PM H.J. Lu wrote:
>
> -mforce-indirect-call generates invalid instruction in 32-bit MI thunk
> since there are no available scratch registers in 32-bit PIC mode.
> Disable -mforce-indirect-call for PIC in 32-bit mode when generating
> MI thunk.
>
> gcc/
>
> PR
This looks like the largest remaining block of related issues...
Gerald
---
htdocs/readings.html | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/htdocs/readings.html b/htdocs/readings.html
index 5e3db8c2..295cc51f 100644
--- a/htdocs/readings.html
+++
This is nearly the last case in our tree; nearly current HTML 5 across,
now.
Gerald
---
htdocs/git.html | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/htdocs/git.html b/htdocs/git.html
index 4a8b1004..fc30c5ab 100644
--- a/htdocs/git.html
+++ b/htdocs/git.html
@@ -3,8 +3
-mforce-indirect-call generates invalid instruction in 32-bit MI thunk
since there are no available scratch registers in 32-bit PIC mode.
Disable -mforce-indirect-call for PIC in 32-bit mode when generating
MI thunk.
gcc/
PR target/105980
* config/i386/i386.cc (x86_output_mi_thunk
On Mon, 16 Jan 2023 at 17:47, François Dumont wrote:
>
> On 13/01/23 17:33, Jonathan Wakely wrote:
> > On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++
> > wrote:
> >> I just rebased this patch.
> >>
> >> All good apart from the to_chars/from_chars symbols issue.
> >>
> >> François
> >
On 13/01/23 18:15, Jonathan Wakely wrote:
@@ -396,7 +376,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
// Non-inline namespace for components replaced by alternates in active
mode.
namespace __cxx1998
{
-# if _GLIBCXX_USE_CXX11_ABI
+# if _GLIBCXX_USE_CXX11_ABI && ! _GLIBCXX_VERSION_NAMESPACE
On 13/01/23 18:06, Jonathan Wakely wrote:
On Fri, 13 Jan 2023 at 16:33, Jonathan Wakely wrote:
On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++
wrote:
I just rebased this patch.
All good apart from the to_chars/from_chars symbols issue.
François
On 11/10/22 19:28, François Dumon
On 13/01/23 17:33, Jonathan Wakely wrote:
On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++
wrote:
I just rebased this patch.
All good apart from the to_chars/from_chars symbols issue.
François
On 11/10/22 19:28, François Dumont wrote:
Hi
Now that pretty printer is fixed (o
Hi,
this patch fixes wrong code issues in ipa-sra where we are trying to prove that
on every execution of a given function a call to other function will happen.
The code uses post dominators and makes a wrong query (which passes only for
first BB in function). Hoever post-dominators are only valid
updated patch in attachments.
diff --git a/libiberty/lrealpath.c b/libiberty/lrealpath.c
index 3c7053b0b70..8858619e854 100644
--- a/libiberty/lrealpath.c
+++ b/libiberty/lrealpath.c
@@ -68,8 +68,138 @@ extern char *canonicalize_file_name (const char *);
/* cygwin has realpath, so it won't get
Tested x86_64-linux. Pushed to trunk.
-- >8 --
When building src/c++20/tzdb.cc we currently get a build error for
--with-default-libstdcxx-abi=gcc4-compatible because std::chrono::tzdb
and related types are not declared for the gcc4-compatible ABI (unless
--disable-libstdcxx-dual-abi is also used
On Mon, Jan 16, 2023 at 09:05:38PM +0800, Kewen.Lin wrote:
> > The *_ok things should only be used for features that can be disabled
> > during configuration, or features that we *want* users to be able to
> > turn off (like FP, VMX, VSX, or HMT or QP float, that kind of thing).
> > That gives quit
From: Eric Botcazou
gcc/ada/
* exp_ch3.adb (Make_Allocator_For_Return): Fix typo in comment.
Tested on x86_64-pc-linux-gnu, committed on master.
---
gcc/ada/exp_ch3.adb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/ada/exp_ch3.adb b/gcc/ada/exp_ch3.adb
index
From: Eric Botcazou
When the type of the return object is a constrained array, there may be an
implicit sliding that needs to be preserved during the expansion.
gcc/ada/
* exp_ch3.adb (Make_Allocator_For_Return): Convert the expression
to the return object's type in the constrai
gcc/ada/
* gcc-interface/Make-lang.in: Update copyright years.
* gcc-interface/Makefile.in: Likewise.
* gcc-interface/ada-builtin-types.def: Likewise.
* gcc-interface/ada-builtins.def: Likewise.
* gcc-interface/ada-tree.def: Likewise.
* gcc-interface
From: Eric Botcazou
The recent removal of the unconditional call to Remove_Side_Effects on the
expression of an object declaration or an allocator with a class-wide type
has introduced a pessimization in the former case for function calls that
return a specific tagged type, because the object ult
From: Eric Botcazou
gcc/ada/
* exp_util.adb (Make_CW_Equivalent_Type.Has_Tag_Of_Type): Fix pasto.
Tested on x86_64-pc-linux-gnu, committed on master.
---
gcc/ada/exp_util.adb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/ada/exp_util.adb b/gcc/ada/exp_util.ad
From: Eric Botcazou
This needs to be done for all expressions with class-wide type.
gcc/ada/
* exp_ch3.adb (Make_Allocator_For_Return): Put back an interface
conversion for expressions with non-interface class-wide type.
Tested on x86_64-pc-linux-gnu, committed on master.
---
From: Eric Botcazou
This further optimizes the usual case of (class-wide) interface objects that
are initialized with calls to functions whose result type is the type of the
objects (this is not necessary as any result type implementing the interface
would do) by avoiding a back-and-forth displac
From: Eric Botcazou
This extends the use of static references to the interface tag in more cases
for (class-wide) interface objects, e.g. for initialization expressions that
are qualified aggregates or nondispatching calls returning a specific tagged
type implementing the interface.
gcc/ada/
From: Eric Botcazou
This restores the proper finalization of temporaries for interface objects
in the case where the initializing expression is not of an interface type.
It turns out that neither Is_Temporary_For_Interface_Object nor its previous
incarnation are sufficient to catch all the vario
From: Eric Botcazou
It turns out that the only blocking case is an aliased object whose nominal
subtype is an unconstrained array because the bounds must be allocated.
gcc/ada/
* exp_ch3.adb (Expand_N_Object_Declaration): Also optimize aliased
objects if their nominal subtype is
From: Eric Botcazou
This optimizes the implementation of (class-wide) interface objects that are
initialized with function calls, by avoiding an unnecessary copy operation.
This also removes useless access checks generated by the expansion of return
statements involving class-wide types.
gcc/ada
Ping David.
Le jeu. 5 janv. 2023 à 23:37, Guillaume Gomez
a écrit :
> Ping David.
>
> Le sam. 24 déc. 2022 à 21:01, Guillaume Gomez
> a écrit :
>
>> Ping David
>>
>> Le jeu. 15 déc. 2022 à 11:34, Guillaume Gomez
>> a écrit :
>>
>>> Forgot it indeed, thanks for notifying me!
>>>
>>> I modified
hello,
I just finished with https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108350
could anyone review and apply the attached patch please?
GCC master was succesfully bootstrapped as x86_64-w64-mingw32.
best!
diff --git a/libiberty/lrealpath.c b/libiberty/lrealpath.c
index 3c7053b0b70..885861
Erm, ignore this - I just rediscovered the approval in a different mail
folder. I forgot that Outlook's automatic email dedpulication meant that
messages CC'd to me end up in one of two different folders at random
when I want them in both.
On Mon, Jan 16, 2023 at 02:03:29PM +, Andrew Carlotti
Hi Richard
I accidentally pushed this patch earlier in the mistaken belief that
you'd already approved it. It looks uncontroversial to me - it just adds
IFN support to build_popcount_expr, analogous to the changes you
suggested and approved for build_cltz_expr (and adjusts testcases
accordingly).
After discussing with Ju.Zhe, we decided to rearrange the pass of vsetvl
insertion instead of call run_fast_dce within this pass.
On Tue, Jan 10, 2023 at 7:21 AM wrote:
> From: Ju-Zhe Zhong
>
> gcc/ChangeLog:
>
> * config/riscv/riscv-vsetvl.cc (pass_vsetvl::done): Add DCE.
> * c
Hi Richard,
> Don't add crtfastmath.o for -shared to avoid altering the FP
> environment when loading a shared library.
>
> PR target/55522
> * config/sol2.h (ENDFILE_SPEC): Don't add crtfastmath.o for -shared.
tested on i386-pc-solaris2.11 and sparc-sun-solaris2.11.
Ok.
Thanks.
Hi Jonathan,
> On Fri, 13 Jan 2023 at 16:39, Jonathan Wakely wrote:
>>
>> On Fri, 13 Jan 2023 at 15:08, Rainer Orth wrote:
>> >
>> > Hi Jonathan,
>> >
>> > > The , , and headers use
>> > > std::errc constants, but don't use std::system_error itself. They only
>> > > use the __throw_system_error(i
Hi,
As Andrew pointed out in PR108396, there is one typo in
rs6000-overload.def on built-in function vec_vsubcuq:
[VEC_VSUBCUQ, vec_vsubcuqP, __builtin_vec_vsubcuq]
"vec_vsubcuqP" should be "vec_vsubcuq", this typo caused
us to define vec_vsubcuqP in rs6000-vecdefines.h instead
of vec_vsubcuq,
Hi Segher!
on 2023/1/16 18:40, Segher Boessenkool wrote:
> Hi!
>
> On Mon, Jan 16, 2023 at 05:20:56PM +0800, Kewen.Lin wrote:
>> on 2023/1/16 16:49, Segher Boessenkool wrote:
+/* { dg-require-effective-target powerpc_p9modulo_ok } */
>>>
>>> Please use a saner selector? If one doesn't exist
On 1/12/23 19:57, Jakub Jelinek via Gcc-patches wrote:
> On Tue, Jan 10, 2023 at 04:33:59PM +, Wilco Dijkstra via Gcc-patches
> wrote:
>> Hi Szabolcs,
>>
>>> i would keep the assert: how[reg] must be either UNSAVED or UNDEFINED
>>> here, other how[reg] means the toggle cfi instruction is mixed
On 1/14/23 22:36, Jan Hubicka wrote:
>> Hi.
>>
>> Noticed during building of libbackend.a with the LTO partial linking.
>>
>> The function release_body is called even if clone_of is a clone
>> of a another function and thus it shares tree declaration. We should
>> preserve it in that situation.
>>
Status
==
The GCC development branch which will become GCC 13 is now in
regression and documentation fixing mode (Stage 4) until we
reach zero P1 regressions and branch for the release.
Quality Data
Priority # Change from last report
--- -
Pushed to wwwdocs.
-- >8 --
---
htdocs/gcc-13/porting_to.html | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/htdocs/gcc-13/porting_to.html b/htdocs/gcc-13/porting_to.html
index 7a67c0b0..9a9a3147 100644
--- a/htdocs/gcc-13/porting_to.html
+++ b/htdocs/gcc-13
Hi!
On Mon, Jan 16, 2023 at 05:20:56PM +0800, Kewen.Lin wrote:
> on 2023/1/16 16:49, Segher Boessenkool wrote:
> >> +/* { dg-require-effective-target powerpc_p9modulo_ok } */
> >
> > Please use a saner selector? If one doesn't exist yet, make a new one?
> > Something that just says "p9", not "mo
Hi!
Ignore __builtins.di like object.d is already ignored.
Committed as obvious.
The modula 2 issue remains...
2023-01-16 Jakub Jelinek
PR other/108413
* update-copyright.py (LibPhobosFilter): Add __builtins.di to
skipped files.
--- contrib/update-copyright.py
+++ co
Pushed to trunk. Backport to gcc-12 for follow (for the first two parts
of the change, tzdb.cc isn't on the branch).
-- >8 --
libstdc++-v3/ChangeLog:
PR libstdc++/108413
* include/c_compatibility/stdatomic.h: Change copyright line to
be consistent with other headers contr
Committed as obvious.
-- >8 --
gcc/ChangeLog:
* doc/extend.texi (Common Function Attributes): Fix grammar.
---
gcc/doc/extend.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index 1103e9936f7..867ef07ea4b 100644
--- a/gc
Hi!
These 2 files had incorrectly formatted Copyright lines (no space between
Copyright and (C)) which makes update-copyright.py upset.
Tested on x86_64-linux with update-copyright --this-year, committed to trunk
as obvious.
2023-01-16 Jakub Jelinek
PR other/108413
* config/r
On Mon, 16 Jan 2023, Jakub Jelinek wrote:
> Hi!
>
> As mentioned on IRC or in PR108413, the last update-copyright.py --this year
> failed and that is why we are in a strange state where some copyrights have
> been updated and others have not.
> The full list of errors I got was I think:
> gcc/m2/
Hi!
As mentioned on IRC or in PR108413, the last update-copyright.py --this year
failed and that is why we are in a strange state where some copyrights have
been updated and others have not.
The full list of errors I got was I think:
gcc/m2/mc-boot/GmcOptions.c: unrecognised copyright: comment (f,
Hi,
Now we will check optimize_function_for_speed_p (cfun) for
TARGET_SAVE_TOC_INDIRECT if it's implicitly enabled. But
the effect of -msave-toc-indirect is actually to save the
TOC in the prologue for indirect calls rather than inline,
it's also good for optimize_function_for_size? So this
patc
Hi Segher,
Thanks for the review comments!
on 2023/1/16 16:49, Segher Boessenkool wrote:
> Hi!
>
> On Mon, Jan 16, 2023 at 04:33:36PM +0800, Kewen.Lin wrote:
>> PR108348 shows one special case that MMA opaque types are
>> used in function arguments and treated as pass by reference,
>> it results
Hi,
As Honza pointed out in [1], the current uses of function
optimize_function_for_speed_p in rs6000_option_override_internal
are too early, since the query results from the functions
optimize_function_for_{speed,size}_p could be changed later due
to profile feedback and some function attributes
Hi!
On Mon, Jan 16, 2023 at 04:33:36PM +0800, Kewen.Lin wrote:
> PR108348 shows one special case that MMA opaque types are
> used in function arguments and treated as pass by reference,
> it results in one copying from argument to a temp variable,
> since this copying happens before rs6000_functio
Hi,
PR108348 shows one special case that MMA opaque types are
used in function arguments and treated as pass by reference,
it results in one copying from argument to a temp variable,
since this copying happens before rs6000_function_arg check,
it can cause ICE without MMA support then. This patc
On Mon, Jan 16, 2023 at 08:19:25AM +0100, Richard Biener wrote:
> + // If op1 and op2 are equivalences, then we don't need a complete cross
> + // product, just pairs of matching elements.
> + if (relation_equiv_p (rel) && lh == rh && num_lh <= 16)
> +{
> + int_range_max tmp;
> + r
Hi Suwa-san,
On Sun, Jan 15, 2023 at 6:53 PM Takayuki 'January June' Suwa
wrote:
>
> In the case of the CALL0 ABI, values that must be retained before and
> after function calls are placed in the callee-saved registers (A12
> through A15) and referenced later. However, it is often the case that
on 2023/1/6 17:26, Kewen.Lin via Gcc-patches wrote:
> Hi,
>
> As PR108272 shows, there are some invalid uses of MMA opaque
> types in inline asm statements. This patch is to teach the
> function rs6000_opaque_type_invalid_use_p for inline asm,
> check and error any invalid use of MMA opaque types
Hi Gerald,
On 14.01.23 22:47, Gerald Pfeifer wrote:
I made a couple of incremental edits. See below for what I just pushed
(and please speak up if you see any issues).
commit 2f870cba58c81449beb618a9030824360a25
...
--- a/htdocs/gcc-13/changes.html
+++ b/htdocs/gcc-13/changes.html
@@ -
67 matches
Mail list logo
