On 11/14/24 4:04 PM, Jan Hubicka wrote:
Hi,
this patch refactors slightly cp/decl.cc so code producing new and delete
operators is not duplicated multiple times. I also broke out
cxx_init_operator_new_delete_decls which I originally inteded to use for
builtion_operator_new/delete which is solved
Hi,
this patch refactors slightly cp/decl.cc so code producing new and delete
operators is not duplicated multiple times. I also broke out
cxx_init_operator_new_delete_decls which I originally inteded to use for
builtion_operator_new/delete which is solved better by Jakub's patch, but it
seems to
Now committed as r15-3799-gcdb9aa0f623ec7 /
https://gcc.gnu.org/r15-3799-gcdb9aa0f623ec7
Tobias
Am 21.09.24 um 01:33 schrieb Tobias Burnus:
Hi Thomas, hello all,
the attached follow-up patch does:
* It fixes an issue (thinko) related to Fortran and \0 terminated,
which fails for at least s
e a
fallback, if needed.
I now added a "return NULL" which clashes a bit with the topic of
caching in the plugin as now the OMP_DEV_%d string also needs to be
cached, but on the libgomp.so side.
* * *
+ sprintf (str, "GPU-%02x" "%02x" "%02x" "%02x
From: Marc Poulhiès
gcc/rust/ChangeLog:
* rust-lang.cc (grs_langhook_type_for_mode): simplify code for
xImode. Add missing long_double_type_node.
Signed-off-by: Marc Poulhiès
---
gcc/rust/rust-lang.cc | 39 +++
1 file changed, 15 insertions(
Changes since V1:
Whitespace fixes noted by the linter
Missed using the iterator for the output template in
_mask pattern!
--
This patch introduces a bit_optab iterator that maps IOR/XOR to bset and
binv (and one day bclr if we need it). That allows us to combine some
patterns that onl
On Tue, 18 Jun 2024, Jeff Law wrote:
> This has gone through my tester. I'll wait for a verdict from pre-commit CI
> before moving forward.
Why do these "[to-be-committed]" annotations end up in the repository
though? It does not appear to me to be useful information to be stored
there forev
This patch introduces a bit_optab iterator that maps IOR/XOR to bset and
binv (and one day bclr if we need it). That allows us to combine some
patterns that only differed in the RTL opcode (IOR vs XOR) and in the
name/assembly (bset vs binv).
Additionally this also allow us to use the iterato
Consider Signed-off-by lines as part of the ending of the initial
commit to avoid having these in the middle of the log when the
changelog part is injected after.
This is particularly usefull with:
$ git gcc-commit-mklog --amend -s
that can be used to create the changelog and add the Signed-off
Richard Sandiford writes:
>> +# this regex matches the first line of the "end" in the initial commit
>> message
>> +FIRST_LINE_OF_END_RE = re.compile('(?i)^(signed-off-by:|co-authored-by:|#)
>> ')
>
> Personally I think it would be safer to drop the final space in the regexp.
>
> OK with that
Marc Poulhiès via Gcc-patches writes:
> Richard Sandiford via Gcc-patches writes:
>>> +# this regex matches the first line of the "end" in the initial commit
>>> message
>>> +FIRST_LINE_OF_END_RE = re.compile('(?i)^(signed-off-by|co-authored-by|#):
>>> ')
>>
>> The current code only requires an
Richard Sandiford via Gcc-patches writes:
>> +# this regex matches the first line of the "end" in the initial commit
>> message
>> +FIRST_LINE_OF_END_RE = re.compile('(?i)^(signed-off-by|co-authored-by|#): ')
>
> The current code only requires an initial "#", rather than an initial "#: ".
> Is th
Marc Poulhiès via Gcc-patches writes:
> Consider Signed-Off-By lines as part of the ending of the initial
> commit to avoid having these in the middle of the log when the
> changelog part is injected after.
>
> This is particularly usefull with:
>
> $ git gcc-commit-mklog --amend -s
>
> that can
Consider Signed-off-by lines as part of the ending of the initial
commit to avoid having these in the middle of the log when the
changelog part is injected after.
This is particularly usefull with:
$ git gcc-commit-mklog --amend -s
that can be used to create the changelog and add the Signed-off
Consider Signed-Off-By lines as part of the ending of the initial
commit to avoid having these in the middle of the log when the
changelog part is injected after.
This is particularly usefull with:
$ git gcc-commit-mklog --amend -s
that can be used to create the changelog and add the Signed-Off
Consider Signed-off-by lines as part of the ending of the initial
commit to avoid having these in the middle of the log when the
changelog part is injected after.
This is particularly usefull with:
$ git gcc-commit-mklog --amend -s
that can be used to create the changelog and add the Signed-Off
On 9/30/22 18:07, H.J. Lu wrote:
On Fri, Sep 30, 2022 at 4:06 PM Jeff Law wrote:
It's a bit weird that free_dom_edge_info leaves a dangling pointer in
e->aux. Not sure what I was thinking.
There's two callers. One wipes e->aux immediately after the call, the
other attaches a newly created
On Fri, Sep 30, 2022 at 4:06 PM Jeff Law wrote:
>
>
> It's a bit weird that free_dom_edge_info leaves a dangling pointer in
> e->aux. Not sure what I was thinking.
>
>
> There's two callers. One wipes e->aux immediately after the call, the
> other attaches a newly created object immediately afte
n't currently exist for a given edge.
Bootstrapped and regression tested on x86_64. Installing on the trunk.
Jeff
commit fbd95c027edcc169cc3b40806375fbabc08500e0
Author: Jeff Law
Date: Fri Sep 30 18:59:24 2022 -0400
Minor cleanup/prep in DOM
It's a bit weird tha
esellschaft: München; Registergericht
München, HRB 106955
commit f62156eab7b757d1ee03a11d5c96c72bd3de079c
Author: Tobias Burnus
Date: Fri Feb 4 17:31:21 2022 +0100
libgomp.fortran/allocate-1.f90: Fix minor cleanup
libgomp/ChangeLog:
* testsuite/libgomp.fortran/allocate-1.
have now _finally_ committed the patch.
>
> Attached is the simplified (-w) diff, where I did exclude the
> indentation changes to make the diff more readable.
>
> For the full diff, see e.g. https://gcc.gnu.org/r12-7053
>
> Tobias
> commit 6d4981350168f1eb3f72149bd7e05b
; Geschäftsführer: Thomas
Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht
München, HRB 106955
commit 6d4981350168f1eb3f72149bd7e05b9ba6bec1fd
Author: Tobias Burnus
Date: Fri Feb 4 14:51:01 2022 +0100
libgomp.fortran/allocate-1.f90: Minor cleanup
This gets rid of the DECL_STUBBED_P macro and adjusts Call_to_gnu.
Tested on x86_64-pc-linux-gnu, committed on trunk
gcc/ada/
* gcc-interface/ada-tree.h (DECL_STUBBED_P): Delete.
* gcc-interface/decl.c (gnat_to_gnu_entity): Do not set it.
* gcc-interface/trans.c (Call_to_
On 9/10/20 11:22 AM, Jakub Jelinek wrote:
But it is not mapnum > 0 here but mapnum >1
Error: need more coffee (well, tea)
Thanks for both the reasoning and catching this oversight of mine.
Tobias
-
Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
On Thu, Sep 10, 2020 at 11:17:34AM +0200, Tobias Burnus wrote:
> Hi Jakub, hello all,
>
> when looking at target.c, I stumbled over that code:
> size_t mapnum → unsigned
> if (mapnum == 0)
> ...
> return;
>
> if (mapnum > 0 || )
> ...
But it is not mapnum > 0 here but mapn
d the if-body unconditionally,
removing some now pointless assignments.
OK?
Tobias
-
Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander
Walter
libgomp/target.c: Minor cleanup
libg
This makes a couple of changes in the code dealing with a call to a
derived subprogram: first, it removes useless calls to Relocate_Node
before calls to OK_Convert_To and Unchecked_Convert_To (these functions
already call Relocate_Node on their second operand) and merge calls to
Analyze and Resolve
On 9/20/18 6:51 AM, Richard Biener wrote:
> On Mon, Sep 17, 2018 at 4:50 PM Jeff Law wrote:
>>
>> This is a relatively minor cleanup that I should have caught last cycle,
>> but somehow missed.
>>
>> We have two structures TO_REMOVE_EDGES and TO_UPDATE_SWITCH_S
On Mon, Sep 17, 2018 at 4:50 PM Jeff Law wrote:
>
> This is a relatively minor cleanup that I should have caught last cycle,
> but somehow missed.
>
> We have two structures TO_REMOVE_EDGES and TO_UPDATE_SWITCH_STMTS which
> are used by the VRP/EVRP code to record edges to
This is a relatively minor cleanup that I should have caught last cycle,
but somehow missed.
We have two structures TO_REMOVE_EDGES and TO_UPDATE_SWITCH_STMTS which
are used by the VRP/EVRP code to record edges to remove and switch
statements that need updating.
They are currently implemented as
This replaces the scanning of the -fstack-usage output with -Wstack-usage.
Tested on x86-64/Linux, applied on the mainline and 8 branch.
2018-05-31 Eric Botcazou
* gnat.dg/stack_usage1.adb: Replace -fstack-usage with -Wstack-usage.
* gnat.dg/stack_usage1b.adb: Likewise.
This factors out the various cases where a marker for an unknown value is
output by the -gnatR switches. No functional changes.
Tested on x86_64-pc-linux-gnu, committed on trunk
2018-05-29 Eric Botcazou
gcc/ada/
* repinfo.adb (Write_Unknown_Val): New procedure.
(List_GCC_Exp
This removes the Truth_Andif_Expr and Truth_Orif_Expr codes for expressions
handled by the repinfo unit, since they are redundant with Truth_And_Expr
and Truth_Or_Expr respectively in this context. No functional changes.
Tested on x86_64-pc-linux-gnu, committed on trunk
2018-05-28 Eric Botcazou
The inter-unit inlining done by the compiler requires a dedicated machinery
to deal with the public status of library-level entities, since it breaks
the private/plublic semantic barrier of the language.
This is a minor cleanup to this machinery, no functional changes.
Tested on x86_64-pc-linux
No need to use a dedicated variable if we can use RECORD_OR_UNION_TYPE_P
directly and only once.
Bootstrapped/regtested on x86_64-linux, applying to trunk.
2017-07-25 Marek Polacek
* c-decl.c (grokfield): Remove local variable.
diff --git gcc/c/c-decl.c gcc/c/c-decl.c
index 50da185e3
On Fri, May 20, 2016 at 03:23:49PM -0600, Jeff Law wrote:
> On 05/19/2016 05:11 PM, Jeff Law wrote:
> [ ... ]
> >This is a bit of a mess and I think the code
> >needs some TLC before we start hacking it up further.
> >
> >Let's start with clean up of dead code:
> >
> > /* We will need to ensure tha
On Fri, May 20, 2016 at 03:23:49PM -0600, Jeff Law wrote:
> On 05/19/2016 05:11 PM, Jeff Law wrote:
> [ ... ]
> >This is a bit of a mess and I think the code
> >needs some TLC before we start hacking it up further.
> >
> >Let's start with clean up of dead code:
> >
> > /* We will need to ensure tha
> I pondered that as a direction, but was scared off by the overall
> fragility of this code when I looked back through the old BZs. I
> figured cleanup preserving existing behavior was the first step.
Setting aside the flag_split_stack stuff, the must_align logic is clear enough
and quite local
On 05/20/2016 03:44 PM, Eric Botcazou wrote:
So here's that cleanup. The diffs are larger than one might expect
because of the reindentation that needs to happen. So I've included a
-b diff variant which shows how little actually changed here.
I'm wondering if it isn't counter-productive. Th
> So here's that cleanup. The diffs are larger than one might expect
> because of the reindentation that needs to happen. So I've included a
> -b diff variant which shows how little actually changed here.
I'm wondering if it isn't counter-productive. The ??? comment is explicit
about where the
On 05/19/2016 05:11 PM, Jeff Law wrote:
[ ... ]
This is a bit of a mess and I think the code
needs some TLC before we start hacking it up further.
Let's start with clean up of dead code:
/* We will need to ensure that the address we return is aligned to
REQUIRED_ALIGN. If STACK_DYNAMIC_O
This consistently passes NULL_TREE as operand #2 of COMPONENT_REF and #3 of
ARRAY_REF/ARRAY_RANGE_REF in gigi. There is no functional change since we
never build them with these operands in the first place.
Tested on x86_64-suse-linux, applied on the mainline.
2016-05-02 Eric Botcazou
This is a small refactoring of the handling of ranges of values.
Tested on x86_64-suse-linux, applied on the mainline.
2016-05-02 Eric Botcazou
* gcc-interface/trans.c (Range_to_gnu): New static function.
(Raise_Error_to_gnu) : Call it to translate the range.
(gnat_to
This just moves the VIS3 mulx patterns to a more appropriate place.
Tested on SPARC/Solaris, applied on the mainline.
2015-11-20 Eric Botcazou
* config/sparc/sparc.md (umulxhi_vis): Move around.
(*umulxhi_sp64): Likewise.
(umulxhi_v8plus): Likewise.
(xmulx_vis
This removes a couple of redundant/unused things. No functional changes.
Tested on x86_64-pc-linux-gnu, committed on trunk
2015-10-16 Eric Botcazou
* a-tags.ads (Parent_Size): Remove obsolete pragma Export.
* s-finmas.ads (Header_Offset): Delete.
* s-finmas.adb (Heade
n the trunk.
Jeff
commit e1eb08886003fdc954c5763ec712109e158f1b0c
Author: Jeff Law
Date: Wed Oct 14 13:01:50 2015 -0400
[PATCH] [PR testsuite/67959]Minor cleanup for ssa-thread-13.c
PR testsuite/67959
* gcc.dg/tree-ssa/ssa-thread-13.c: Avoid bitfield assump
I've been meaning to push this for a while. Primarily it removes two
set but never used private fields from the class and simplifies the
constructor in the obvious way. It also fixes a couple minor formatting
glitches, adds some missing comments, etc. It's ultimately related to
getting the
I've checked in the attached patch to clean up a few stylistic issues in
the nios2 back end, e.g. switching to the new "rtx_insn *" type instead
of "rtx" for things that are explicitly insns. There's no change in
functionality.
-Sandra
2015-07-04 Sandra Loosemore
gcc/
* config/nios2/ni
This fixes a few typos and formatting glitches, i.e. no functional change.
Tested on x86_64-suse-linux, applied on the mainline as obvious.
2015-06-25 Eric Botcazou
* lto-streamer-out.c (DFS::hash_scc): Fix typos and formatting glitches.
--
Eric BotcazouIndex: lto-streamer-out.c
=
Tested on x86_64-suse-linux, applied on the mainline.
2015-05-25 Eric Botcazou
* get_targ.adb (C_Get_Float_Words_BE): Adjust import name.
(C_Get_Words_BE): Likewise.
(C_Get_Bytes_BE): Likewise.
(C_Get_Bits_BE): Likewise.
(Width_From_Size): Remove superf
Tested on x86_64-suse-linux, applied on the mainline.
2015-05-25 Eric Botcazou
* gcc-interface/decl.c (elaborate_expression): Take a const string
suffix instead of a tree.
(elaborate_expression_1): Likewise.
(elaborate_expression_2): Likewise.
(gnat_to_
Tested on x86_64-suse-linux, applied on the mainline.
2015-05-25 Eric Botcazou
* gcc-interface/decl.c (gnat_to_gnu_entity) : Move around.
: Merge with E_Variable case.
: Remove 'object' label.
--
Eric BotcazouIndex: gcc-interface/decl.c
=
Tested on x86_64-suse-linux, applied on the mainline.
2015-05-25 Eric Botcazou
* gcc-interface/decl.c (gnat_to_gnu_entity): Minor tweak.
* gcc-interface/trans.c (finalize_nrv_unc_r): Use CONSTRUCTOR_ELT.
* gcc-interface/utils.c (convert): Likewise and simplify.
> 2015-04-23 Steve Ellcey
>
> * config/mips/mips.md: (*madd4) Remove accum_in attribute.
> (*madd3): Ditto.
> (*msub4): Ditto.
> (*msub3): Ditto.
> (*nmadd4): Ditto.
> (*nmadd3): Ditto.
> (*nmadd4_fastmath): Ditto.
> (*nmadd3_fastmath): Ditto.
>
This is a minor cleanup patch for MIPS. A number of floating point madd type
instructions set the accum_in attribute. But this attribute is only used
for integer madd instructions, so this patch removes it from the floating
point madd instructions where it is not needed or used.
The 'acc
Applied, thanks.
Jason
On 14 September 2014 21:36, Paolo Carlini wrote:
> Hi,
>
> On 09/14/2014 07:44 PM, Ville Voutilainen wrote:
>>
>> Tested on Linux-x64.
>
> Thanks. In such cases you can as well remove the main: then is also
> immediately obvious the the testcase is a dg-do compile.
Sure, but sometimes I use such
Hi,
On 09/14/2014 07:44 PM, Ville Voutilainen wrote:
Tested on Linux-x64.
Thanks. In such cases you can as well remove the main: then is also
immediately obvious the the testcase is a dg-do compile.
Paolo.
Tested on Linux-x64.
/testsuite
2014-09-14 Ville Voutilainen
Minor cleanup, don't run tests for which compilation is enough.
* g++.dg/g++.dg/cpp1y/var-templ1.C: Change return to
static_assert, compile instead of running.
* g++.dg/g++.dg/cpp1y/var-templ3.C: Likewise.
* g++
Dear Paul,
> OK for trunk.
Thanks for the review. Committed as r212330.
Dominique
Dear Dominique,
OK for trunk.
Thanks for the patch
Paul
On 5 July 2014 18:09, Dominique Dhumieres wrote:
> On Mon, 05 May 2014, I have posted
> https://gcc.gnu.org/ml/fortran/2014-05/msg00012.html.
> On IRC Tobias Burnus remarked that the *.mod file in
> gfortran.dg/vect/fast-math-real8-pr4
On Mon, 05 May 2014, I have posted
https://gcc.gnu.org/ml/fortran/2014-05/msg00012.html.
On IRC Tobias Burnus remarked that the *.mod file in
gfortran.dg/vect/fast-math-real8-pr40801.f90
should be cleaned automatically. This is not done because the cleaning is done
in gfortran-dg-runtest and not
There was apparently a last-minute name change for DECL_NONALIASED.
Tested on x86_64-suse-linux, applied on mainline and 4.9 branch as obvious.
2014-06-12 Eric Botcazou
* tree-core.h (DECL_NONALIASED): Use proper spelling in comment.
--
Eric BotcazouIndex: tree-core.h
This exports End_Location from sinfo and uses it in gigi, instead of redoing
the computation locally.
Tested on x86_64-suse-linux, applied on the mainline.
2014-05-18 Eric Botcazou
* fe.h (Set_Present_Expr): Move around.
(End_Location): New macro.
* gcc-interface/tra
This replaces an explicit test for private types by Underlying_Type, which
does the test automatically.
Tested on x86_64-suse-linux, applied on the mainline.
2014-05-18 Eric Botcazou
* gcc-interface/decl.c (gnat_to_gnu_entity): Use Underlying_Type in
lieu of more verbose con
If no one objects, I'll commit the following patch in a couple of days
2014-05-05 Dominique d'Humieres
* gfortran.dg/list_read_12.f90: Delete the file.
* gfortran.dg/gfortran.dg/vect/fast-math-real8-pr40801.f90:
Cleanup module yomphy0.
--- ../_clean/gcc/testsuite/gfort
This patch ports another* patch from the Fortran-caf branch to the
trunk; it only affects -fcoarray=lib. An earlier patch was posted
before,** remains unreviewed and is replaced by this patch (re-diffed,
minutely enhanced).
Besides some minor clean-up in libgfortran/caf, the patch changes the
This fixes a few glitches introduced by the recent changes to the file.
Tested on x86-64/Linux, applied on the mainline as obvious.
2013-12-17 Eric Botcazou
* expmed.c (lowpart_bit_field_p): Fix comment.
(store_bit_field_using_insv): Fix formatting.
(store_bit_field):
Tested on x86_64-suse-linux, applied on the mainline.
2013-08-13 Eric Botcazou
* gcc-interface/decl.c (gnat_to_gnu_entity): Replace True with true.
(is_cplusplus_method): Likewise, and False with false.
(components_need_strict_alignment): Likewise.
* gcc-interf
On Thu, Sep 13, 2012 at 8:05 PM, Marc Glisse wrote:
> Hello,
>
> this patch is a minor cleanup of my previous forwprop patches for vectors. I
> have known about get_prop_source_stmt from the beginning, but for some
> reason I always used SSA_NAME_DEF_STMT. This makes the sourc
Hello,
this patch is a minor cleanup of my previous forwprop patches for vectors.
I have known about get_prop_source_stmt from the beginning, but for some
reason I always used SSA_NAME_DEF_STMT. This makes the source code
slightly shorter, and together with PR 54565 it should help get some
Ok for google branches (main and 4_7).
thanks,
David
On Wed, Mar 21, 2012 at 2:45 PM, Harshit Chopra wrote:
> 2012-03-21 Harshit Chopra
>
> Minor changes:
> i386.c: made check_should_patch_current_function C90 compatible.
> i386.md: Added '\t' to bytes generated by
> ix86
2012-03-21 Harshit Chopra
Minor changes:
i386.c: made check_should_patch_current_function C90 compatible.
i386.md: Added '\t' to bytes generated by
ix86_output_function_nops_prologue_epilogue for proper formatting
of assembly.
patch-functions-*.c: Fixed
I noticed that gimplification made some code in tree-eh.c unnecessary,
but did not actually remove it. This patch cleans up tree-eh.c to
remove the unnecessary code and simplify the do_return_redirection
function slightly. Bootstrapped and ran testsuite on
x86_64-unknown-linux-gnu. Committed to
This obvious patch just removes unused tree_ssa_loop_version function
declaration from tree-flow.h. Will be committed in 24 hours if no objection.
2011-07-20 Roman Zhuykov
* tree-flow.h (tree_ssa_loop_version): Remove unused
declaration.
---
gcc/tree-flow.h |2 --
1 files
75 matches
Mail list logo
