On 1 December 2015 at 18:28, David Malcolm wrote:
> On Tue, 2015-12-01 at 13:55 +0100, Bernhard Reutner-Fischer wrote:
>> +/* Lookup function FN fuzzily, taking names in FUN into account. */
>> +
>> +const char*
>> +gfc_lookup_function_fuzzy (const
On December 1, 2015 12:05:39 AM GMT+01:00, Jan Hubicka wrote:
>Hi,
>this is first patch in the broken up series. It adds the logic into
>ipa-inline-transform to drop the flag when inlining. I do it always
>until
>we find a way to make early optimizations safe WRT this transform.
>
>The testcase
Trevor,
On 1 May 2015 at 01:23, Trevor Saunders wrote:
> On Thu, Apr 30, 2015 at 11:58:09PM +0200, Bernhard Reutner-Fischer wrote:
>> On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law wrote:
>> >On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
>> >> Hi,
>
On 20 February 2020 01:26:58 CET, Martin Sebor wrote:
+ Sets *NULTREM if the representation contains a zero byte, and sets
s/NULTREM/NULTERM/
thanks,
On Mon, 2 Mar 2020 16:48:26 -0500
David Malcolm wrote:
> +static inline bool
> +is_std_function_p (const_tree fndecl)
> +{
> + tree name_decl = DECL_NAME (fndecl);
> + if (!name_decl)
> +return false;
> + if (!DECL_CONTEXT (fndecl))
> +return false;
> + if (TREE_CODE (DECL_CONTEXT (f
On Wed, 04 Mar 2020 11:16:35 -0500
David Malcolm wrote:
> On Wed, 2020-03-04 at 11:05 -0500, Marek Polacek wrote:
> > On Wed, Mar 04, 2020 at 04:54:54PM +0100, Bernhard Reutner-Fischer
> > wrote:
> > > On Mon, 2 Mar 2020 16:48:26 -0500
> > > David Malcolm
On Mon, 23 Sep 2019 14:52:19 -0500
"Christian Biesinger via gcc-patches" wrote:
> From: Christian Biesinger
>
> Removes an unused include as a cleanup. Requires updating
> lots of files who previously relied on this transitive include.
Note that we have a tool to help prune unused includes, so
On Tue, 24 Sep 2019 12:12:04 +0100
Mark Eggleston wrote:
> >> @@ -411,7 +411,7 @@ gfc_post_options (const char **pfilename)
> >> && flag_max_stack_var_size != 0)
> >> gfc_warning_now (0, "Flag %<-fno-automatic%> overwrites
> >> %<-fmax-stack-var-size=%d%>",
> >> fla
On Tue, 24 Sep 2019 15:11:43 +0100
Mark Eggleston wrote:
> I didn't realise that's how it worked. That's cleaner. Once fixed OK for
> commit?
> +@item -Wno-overwrite-recursive
> +@opindex @code{Woverwrite-recursive}
> +@cindex warnings, overwrite recursive
> +Do not warn when @option{-fno-auto
On 8 October 2019 14:12:47 CEST, Jakub Jelinek wrote:
>On Tue, Oct 08, 2019 at 02:04:17PM +0200, Tobias Burnus wrote:
>> Seemingly, 'target simd' was forgotten – which yielded the error:
>> "Unexpected !$OMP TARGET SIMD statement"
>>
>> OK for the trunk?
>Ok, with moving the test to libgomp.for
On 14 October 2019 10:34:37 CEST, Tobias Burnus wrote:
>In patch https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00092.html , I
>increased the error column location by one to point to the actual
>error.
>
>That works fine, except when the character is one before the end of the
>
>line (= '\0') – as
On 14 October 2019 10:50:55 CEST, Jakub Jelinek wrote:
>On Tue, Oct 08, 2019 at 02:04:17PM +0200, Tobias Burnus wrote:
>> testsuite/
>> * gfortran.dg/gomp/target-simd.f90: New.
>
>John reported in bugzilla the testcase fails on hppa with an error that
>looks like heap corruption, and ind
On 14 October 2019 11:56:28 CEST, Tobias Burnus wrote:
>Hi Bernard,
>
>On 10/14/19 11:52 AM, Bernhard Reutner-Fischer wrote:
>>> Comitted as Rev. 276953 (+ 276955).
>
>The second commit (i.e "+" part) and the second attached patch (scroll
>
>to the las
On 18 October 2019 17:08:54 CEST, Chung-Lin Tang
wrote:
>Also, I've added a new libgomp.oacc-fortran/deviceptr-2.f90 testcase
>that
>actually copies out and verifies the deviceptr computation.
In testcases please do not 'call abort' which is nonstandard but use 'stop N'
which is standard, idea
On 19 October 2019 15:04:39 CEST, Bernhard Reutner-Fischer
wrote:
>On 18 October 2019 17:08:54 CEST, Chung-Lin Tang
> wrote:
>
>>Also, I've added a new libgomp.oacc-fortran/deviceptr-2.f90 testcase
>>that
>>actually copies out and verifies the deviceptr computatio
On Thu, 14 Nov 2019 10:41:25 +0100
Martin Liška wrote:
> On 11/6/19 10:02 PM, Bernhard Reutner-Fischer wrote:
> > Also why do you punt on duplicate conditions like in
> >
> >> +++ b/gcc/testsuite/gcc.dg/tree-ssa/if-to-switch-4.c
> >> +int main(int argc, char
On 16 November 2019 21:33:58 CET, Thomas Koenig wrote:
>Hello world,
>
>here is an update to the patch.
+ char name[GFC_MAX_SYMBOL_LEN + 1];
+ snprintf (name, GFC_MAX_SYMBOL_LEN, "__dummy_%d_%s", var_num++,
+ f->sym->name);
+
+ if (gfc_get_sym_tree (name,
On 19 November 2019 23:54:55 CET, Thomas Koenig wrote:
>Am 19.11.19 um 11:39 schrieb Bernhard Reutner-Fischer:
>> + char name[GFC_MAX_SYMBOL_LEN + 1];
>> + snprintf (name, GFC_MAX_SYMBOL_LEN, "__dummy_%d_%s", var_num++,
>> +f
On Wed, 20 Nov 2019 22:38:30 +0200
Janne Blomqvist wrote:
> On Wed, Nov 20, 2019 at 8:00 PM Bernhard Reutner-Fischer
> wrote:
> >
> > On 19 November 2019 23:54:55 CET, Thomas Koenig
> > wrote:
> > >Am 19.11.19 um 11:39 schrieb Bernhard Reutner-F
On Wed, 1 Aug 2018 at 16:31, Richard Biener wrote:
> --- a/gcc/cfganal.c
> +++ b/gcc/cfganal.c
> @@ -1057,6 +1057,119 @@ pre_and_rev_post_order_compute (int *pre_order, int
> *rev_post_order,
>return pre_order_num;
> }
>
> +/* Unline pre_and_rev_post_order_compute we fill rev_post_order back
On Tue, 16 May 2017 at 21:08, Mike Stump wrote:
>
> On May 16, 2017, at 5:16 AM, Jonathan Wakely wrote:
> > The change I care about in 1.5.3
>
> So, we haven't talked much about the version people want most. If we update,
> might as well get something that more people care about. 1.5.3 is in u
On 7 August 2018 18:34:30 CEST, Segher Boessenkool
wrote:
>On Mon, Aug 06, 2018 at 08:25:49AM -0700, Mike Stump wrote:
>> Since g++ already requires 1.5.3, it make no sense to bump to
>anything older that 1.5.3, so let's bump to 1.5.3. Those packaging
>systems and OSes that wanted to update by n
On 13 August 2018 10:16:20 CEST, Kai Tietz wrote:
>Hello,
>
>this patch implements variable issue rate feature for falkor cpu.
>Additionally this patch adjusts the issue rate for falkor 8 as this
>value reflects more cpu's specification.
>
>This patch was tested against SPEC 2017 & 2016 and showed
On 13 August 2018 15:12:30 CEST, Bernhard Reutner-Fischer
wrote:
>On 13 August 2018 10:16:20 CEST, Kai Tietz
>wrote:
>>Hello,
>>
>>this patch implements variable issue rate feature for falkor cpu.
>>Additionally this patch adjusts the issue rate for falkor 8 as th
On 15 August 2018 18:46:37 CEST, Julian Brown wrote:
>On Mon, 13 Aug 2018 12:06:21 -0700
>Cesar Philippidis wrote:
atttribute has more t than strictly necessary.
Don't like signed integer levels where they should be some unsigned.
Also don't like single switch cases instead of if.
And omitting
On 16 August 2018 17:46:43 CEST, Julian Brown wrote:
>On Wed, 15 Aug 2018 21:56:54 +0200
>Bernhard Reutner-Fischer wrote:
>
>> On 15 August 2018 18:46:37 CEST, Julian Brown
>> wrote:
>> >On Mon, 13 Aug 2018 12:06:21 -0700
>> >Cesar Philippidis wro
On 27 August 2018 14:24:33 CEST, Aldy Hernandez wrote:
>Howdy!
>
>Phew, I think this is the last abstraction. This handles the unary
>CONVERT_EXPR_P code.
+bool
+wide_int_range_convert_tree (tree &tmin, tree &tmax,
+tree outer_type,
+signo
On 30 August 2018 22:20:06 CEST, Marc Glisse wrote:
>Hello,
>
>INTEGRALS_SIGN_PREC_MATCH: the name doesn't really match the semantics.
+#define INTEGRALS_SIGN_PREC_MATCH(A, B) \
+ TYPE_PRECISION (TREE_TYPE (A)) == TYPE_PRECISION (TREE_TYPE (B)) \
+ (TYPE_PRECISION (TREE_TYPE (A)) > TYPE_PRECIS
On Wed, 22 Aug 2018 at 21:37, Janus Weil wrote:
>
> Am Mi., 22. Aug. 2018 um 17:46 Uhr schrieb Thomas Koenig
> :
> >
> > Hi Janus,
> >
> > >> Janus,
> > >>
> > >> On 13 August 2018 21:44:47 CEST, Janus Weil wrote:
> > >>> Hi all,
> > >>>
> > >>> this simple patch improves some of the diagnostics
On Sun, 15 Jul 2018 at 13:20, Thomas Koenig wrote:
> So, here is the final version. I would really like to get this
> into trunk, and out of the way, so Nicolas and I can focus on
> other things.
>
> So, OK?
[I know i'm late as it was already applied]
For me it would be easier to read the locki
On Mon, 3 Sep 2018 at 11:07, Janne Blomqvist wrote:
>
> On Mon, Sep 3, 2018 at 1:39 AM Bernhard Reutner-Fischer
> wrote:
>>
>> On Sun, 15 Jul 2018 at 13:20, Thomas Koenig wrote:
>>
>> + au = (async_unit *) xmalloc (sizeof (async_unit));
>>
>> I
On Tue, 4 Sep 2018 at 18:43, Andrew Benson wrote:
>
> As suggested by Janus, PR87103 is easily fixed by the attached patch which
> increases GFC_MAX_SYMBOL_LEN to 76 (sufficient to hold the maximum allowed F08
> symbol length of 63, plus a null terminator, plus the "__tmp_class_" prefix).
This is
On 7 June 2017 08:44:13 CEST, Jan Hubicka wrote:
>Hi
>the following patch makes cfgbuild to preserve profile when loops are
>introduced at RTL level (not very well, but at least do not throw it
>all
>away) and also avoids re-computing probabilities when there are no
>changes to CFG.
>
>Bootstrappe
On 7 June 2017 16:46:53 CEST, Martin Sebor wrote:
>On 06/07/2017 02:23 AM, Richard Biener wrote:
>> On Wed, Jun 7, 2017 at 5:26 AM, Martin Sebor
>wrote:
Note I'd be _much_ more sympathetic to simply canonicalizing all of
bzero and bcopy
to memset / memmove and be done with all the
On 7 June 2017 19:22:43 CEST, Szabolcs Nagy wrote:
>Current multiarch directory name is always *-linux-gnu* on linux,
>this patch configures different names for uclibc and musl targets.
>(tested by the debian rebootstrap scripts for various *-linux-musl
>and *-linux-uclibc targets see debian bug #
On 8 June 2017 14:52:49 CEST, Jan Hubicka wrote:
>Hi,
>this patch adds static code to detect basic block with 0 execution
>count.
>Those are basic block either reached only by EH or those which leads to
>call of
>function decorated with cold attribute.
>
>Function decorated by noreturn is not suff
On 8 June 2017 17:27:24 CEST, Jan Hubicka wrote:
>Hi,
>this patch adds cold attributes to (some of) our internal checking.
Exactly.
Same for anything touching stderr.
/me hides
On 14 June 2017 at 15:08, Bin Cheng wrote:
> Hi,
> Loop split currently generates below control flow graph for split loops:
> +
> + .-- guard1 --.
> + v v
> + pre1(loop1).-->pre2(loop2)
> + | |
On 21 June 2017 16:07:29 CEST, "Tsimbalist, Igor V"
wrote:
>Thanks for the feedback. I'll redo the patch according to your
>comments.
what is "noni-tracking" ? Surplus i.
"codegeneration" probably lacks a space.
Thanks,
On Fri, Nov 10, 2017 at 02:14:25PM +, Bin.Cheng wrote:
> Hmm, the patch...
+ /* Setup UID for all statements in dominance order. */
+ basic_block *bbs = get_loop_body (loop);
+ for (i = 0; i < loop->num_nodes; i++)
+{
+ unsigned uid = 0;
+ basic_block bb = bbs[i];
+
+ fo
Hi!
Ok for trunk?
gcc/ChangeLog:
2017-11-12 Bernhard Reutner-Fischer
* gdbinit.in: Break on gfc_internal_error.
Signed-off-by: Bernhard Reutner-Fischer
---
gcc/gdbinit.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/gdbinit.in b/gcc/gdbinit.in
index be56b0ee25b
On 13 November 2017 01:52:41 CET, Martin Sebor wrote:
s/^\(he array\)/t\1/
thanks,
On 17 November 2017 14:31:45 CET, Richard Biener
wrote:
>On Fri, Nov 17, 2017 at 11:23 AM, Eric Botcazou
>wrote:
>> Hi,
>>
>> this is a cleaned up and updated revision of Mike's latest posted
>patch
>> implementing #pragma GCC unroll in the C and C++ compilers. To be
>honest,
>> we're not so mu
On 17 November 2017 14:45:29 CET, Richard Biener
wrote:
>On Fri, Nov 17, 2017 at 8:41 AM, Jeff Law wrote:
>> This patch introduces the evrp_range_analyzer class. This is the
>class
>> we're going to be able to embed into existing dominator walkers to
>> provide them with context sensitive range
On 20 November 2017 at 12:26, Eric Botcazou wrote:
>> If anybody finds the time to push the corresponding Fortran changes then I'd
>> be grateful. I won't have time for this until end of stage 1...
>>
>> https://gcc.gnu.org/ml/fortran/2015-02/msg00014.html
>
> OK, I'm going to merge it in the main
On 20 November 2017 at 15:45, Steve Kargl
wrote:
> On Mon, Nov 20, 2017 at 12:57:47PM +0100, Bernhard Reutner-Fischer wrote:
>> On 20 November 2017 at 12:26, Eric Botcazou wrote:
>> >> If anybody finds the time to push the corresponding Fortran changes then
>> >&g
On Mon, Nov 20, 2017 at 08:03:23PM -0500, David Malcolm wrote:
> [1] FWIW the script I use for this is here:
> https://github.com/davidmalcolm/gcc-build
NUM_CORES=$(getconf _NPROCESSORS_ONLN || echo 1)
You usually don't want to count offline processors.
and in create_src, instead of fiddling
On 21 November 2017 20:53:50 CET, Eric Botcazou wrote:
>> This patch by Than McIntosh fixes a small bug in the libgo Makefile
>> recipe that constructs the directory from which to pick up
>> libgcc_s.so; the gccgo invocation with -print-libgcc-file-name was
>> missing the flags, which meant that f
On 22 November 2017 at 09:07, Eric Botcazou wrote:
>> grep -q was even required by SUSv2 from 1997 so Solaris should really
>> support it. What version if Solaris is that and what version of grep?
>
> /usr/bin/grep on Solaris 10 (/usr/xpg4/bin/grep does support it).
Why would we want to use /usr/
On 16 June 2010 at 14:56, Janus Weil wrote:
>>> Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>>>
>>
>> OK. Kudos to Dominique for finding it and thanks, Janus, for fixing it.
>
> Thanks. Committed as r160834.
@@ -4591,12 +4591,14 @@ gfc_check_symbol_typed (gfc_symbol* sym, gfc_namespace*
On 5 December 2017 at 19:02, Bernhard Reutner-Fischer
wrote:
> On 16 June 2010 at 14:56, Janus Weil wrote:
>>>> Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>>>>
>>>
>>> OK. Kudos to Dominique for finding it and thanks, Janus, for
On 12 December 2017 21:15:25 CET, Martin Sebor wrote:
>
>Tested on x86_64-linux.
I assume this test worked even before this patch. Thus:
s/oveflow/overflow/
thanks,
On 13 December 2017 22:30:12 CET, David Malcolm wrote:
>-/* Walk over all statements of all reachable BBs and call
>check_array_bounds
>- on them. */
>+/* A dom_walker subclass for use by vrp_prop::check_all_array_refs,
>+ to walk over all statements of all reachable BBs and call
Not all s
On 25 December 2017 20:45:40 CET, Steve Kargl
wrote:
>The attach patch fixes a problem when a LOGICAL subprogram
>appears as the first element in an array constructor, which
>is interpreted as a LOGICAL type-spec, which fails because
>the argument is of type LOGICAL instead of INTEGER.
>
>Regress
Joseph,
On Sat, Jan 06, 2018 at 08:45:33AM +0100, Richard Biener wrote:
> On January 5, 2018 10:13:34 PM GMT+01:00, Joseph Myers
> wrote:
> >unrelated. OK to commit?
>
> OK.
>
> Richard.
> >Index: gcc/match.pd
> >===
> >--- g
On Wed, Jan 10, 2018 at 08:17:08PM +, Joseph Myers wrote:
> On Wed, 10 Jan 2018, Bernhard Reutner-Fischer wrote:
>
> > > >+/* x <= +Inf is the same as x == x, i.e. !isnan(x), but this
> > > >loses
> > > >+ an "invalid"
On 10 January 2018 15:59:28 CET, "Martin Liška" wrote:
>On 01/10/2018 02:13 PM, Richard Biener wrote:
>> On Tue, Jan 9, 2018 at 7:29 PM, Jeff Law wrote:
>>> On 01/09/2018 07:43 AM, Martin Liška wrote:
On 09/20/2017 05:00 PM, Jeff Law wrote:
> On 09/20/2017 01:24 AM, Martin Liška wrote:
>
On 23 January 2018 11:31:27 CET, Tamar Christina
wrote:
>Hi All,
>
>This patch makes dg-cmp-results.sh reject the use of log files in the
>comparison.
>Often when given a log file dg-cmp-results will give incomplete/wrong
>output and
>using log instead of sum is one autocomplete tab away.
>
>Inst
On 27 January 2018 12:03:57 CET, Janne Blomqvist
wrote:
>On Sat, Jan 27, 2018 at 1:31 AM, Steve Kargl
> wrote:
>> The attached patch implements a check for F2015:C830.
>> The wording of the F2008:C531 is nearly identical, but
>> the restriction on BLOCK is noted in the normative test.
>> The 3 li
On 29 January 2018 04:26:17 CET, Kito Cheng wrote:
>Hi all:
>
>This patch enable RISC-V support FreeBSD, Ruslan (RISC-V FreeBSD
>maintainer) and me has been tested on FreeBSD 12 for building kernel
>and whole user space programs/libraries.
I think the copyright year of the new header should be 20
On 19 June 2016 at 22:21, Mike Stump wrote:
> On Jun 18, 2016, at 12:31 PM, Bernhard Reutner-Fischer
> wrote:
>>
>> A branch with a name matching scan-assembler pattern triggers
>> inappropriate FAIL.
>
>> The patch below adds -fno-ident if a testcase contai
>This patch adds scan-ltrans-tree-dump.
Please check all error calls to talk about the correct function -- at least
scan-ltrans-tree-dump-times is wrong.
thanks,
>
>Bootstrapped and reg-tested on x86_64.
>
>OK for stage4/stage1 trunk?
>
>Thanks,
>- Tom
On 4 April 2018 at 10:59, Tom de Vries wrote:
> On 04/03/2018 07:49 PM, Bernhard Reutner-Fischer wrote:
>>>
>>> This patch adds scan-ltrans-tree-dump.
>>
>>
>> Please check all error calls to talk about the correct function -- at
>> least sc
On 11 April 2018 13:05:52 CEST, Claudiu Zissulescu
wrote:
>Hi,
>
>Please find the ARC's gcc8 changes entry section as committed to
>wwwdocs.
s/qualifer/qualifier/
thanks,
On Wed, 5 Sep 2018 at 03:30, Jerry DeLisle wrote:
>
> On 09/04/2018 10:43 AM, Bernhard Reutner-Fischer wrote:
> > On Tue, 4 Sep 2018 at 18:43, Andrew Benson
> > wrote:
> >>
> >> As suggested by Janus, PR87103 is easily fixed by the attached patch which
>
On Wed, 5 Sep 2018 at 12:35, Hans-Peter Nilsson
wrote:
>
> These pthread_mutex_t were obviously meant to be
> __gthread_mutex_t. See other declarations. Not being that,
> broke cris-elf build at r264070, restored with this patch. Also
> regtested on native x86_64-pc-linux-gnu.
>
> I'm not sure
From: Bernhard Reutner-Fischer
Aids debugging the fortran FE.
gcc/ChangeLog:
2017-11-12 Bernhard Reutner-Fischer
* gdbinit.in: Break on gfc_internal_error.
---
gcc/gdbinit.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/gdbinit.in b/gcc/gdbinit.in
index 4db977f0bab
-foo-linux.
I'd appreciate if someone could double check for regressions on other
setups. Git branch:
https://gcc.gnu.org/git/?p=gcc.git;a=log;h=refs/heads/aldot/fortran-fe-stringpool
Ok for trunk?
Bernhard Reutner-Fischer (29):
gdbinit: break on gfc_internal_error
Use
From: Bernhard Reutner-Fischer
The openmp part will be cleaned up later in this series.
gcc/fortran/ChangeLog:
2017-10-22 Bernhard Reutner-Fischer
* match.h (gfc_match_defined_op_name): Adjust prototype and add
a parameter USER_OPERATOR.
* matchexp.c
From: Bernhard Reutner-Fischer
Introduce a helper to construct a user operator from a name and the
reverse operation, i.e. a helper to construct a name from a user
operator.
gcc/fortran/ChangeLog:
2017-10-29 Bernhard Reutner-Fischer
* gfortran.h (gfc_get_uop_from_name
From: Bernhard Reutner-Fischer
Use a stringpool-node for those gfc_code2string values that are used as
names.
gcc/fortran/ChangeLog:
2017-10-26 Bernhard Reutner-Fischer
* interface.c (gfc_match_generic_spec, gfc_check_dtio_interfaces,
gfc_find_typebound_dtio_proc
From: Bernhard Reutner-Fischer
2017-10-26 Bernhard Reutner-Fischer
* gfortran.h (struct gfc_association_list): Change name to
pointer.
* match.c (gfc_match_associate): Adjust.
---
gcc/fortran/gfortran.h | 2 +-
gcc/fortran/match.c| 6 ++
2 files changed, 3
From: Bernhard Reutner-Fischer
Ideally we would populate mstrings structs with strings obtained through
the stringpool. Doing so by means of minit wouldn't work out too well
though, see comment in gfortran.h. We could replace the initialized
strings in gfc_init_1 but that's for a l
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-10-29 Bernhard Reutner-Fischer
* gfortran.h (struct gfc_use_rename): Use pointers for
local_name and use_name.
* match.c (gfc_match): Set name to NULL on failed match.
* module.c (gfc_match_use): Use
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-15 Bernhard Reutner-Fischer
* gfortran.h (struct gfc_common_head, struct gfc_intrinsic_arg):
Make name a pointer.
* intrinsic.c (add_sym): Use stringpool for name.
* match.c (gfc_get_common
From: Bernhard Reutner-Fischer
Occurrences of name2 in this patch will be fixed later in this series.
gcc/fortran/ChangeLog:
2017-10-23 Bernhard Reutner-Fischer
* match.h (gfc_match_name): Pass argument by reference. Adjust
all callers.
(match_common_name): Likewise
From: Bernhard Reutner-Fischer
Switch type bound procedures to use the stringpool.
gcc/fortran/ChangeLog:
2017-11-24 Bernhard Reutner-Fischer
* decl.c (gfc_match_decl_type_spec): Use stringpool.
* module.c (mio_expr): Likewise.
(mio_typebound_proc): Likewise
From: Bernhard Reutner-Fischer
Due to a typo a user operator used in a reduction was not found in the
symtree so would have been written multiple times (in theory).
E.g. user operator ".add." was looked up as ".ad" instead of "add".
For gcc-8 branch and earlier on
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-29 Bernhard Reutner-Fischer
* match.c (gfc_get_common): Use stringpool for mangled name.
---
gcc/fortran/match.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/fortran/match.c b/gcc/fortran
From: Bernhard Reutner-Fischer
Copying the sym->name ruins pointer equality checks and first and
foremost is not needed nowadays.
gcc/fortran/ChangeLog:
2018-09-02 Bernhard Reutner-Fischer
* decl.c (gfc_match_volatile, gfc_match_asynchronous): Do not
copy sym-&g
From: Bernhard Reutner-Fischer
Add matched names into the stringpool.
gcc/fortran/ChangeLog:
2017-10-26 Bernhard Reutner-Fischer
* match.c (gfc_match): Use pointer to pointer when matching a
name via "%n" format. Adjust all callers.
(gfc_m
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-29 Bernhard Reutner-Fischer
* iresolve.c (gfc_resolve_ctime_sub): Use stringpool for
intrinsic subroutine name.
(gfc_resolve_fdate_sub): Likewise.
(gfc_resolve_gerror): Likewise
From: Bernhard Reutner-Fischer
This gets rid of some of the str[n]*cmp in favour of (faster) pointer
equality checks.
gcc/fortran/ChangeLog:
2017-11-02 Bernhard Reutner-Fischer
* check.c (gfc_check_move_alloc): Use pointer comparison instead
of strcmp.
* class.c
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-28 Bernhard Reutner-Fischer
* module.c (struct pointer_info): Change binding_label to const
pointer.
(free_pi_tree): Do not free binding_label.
(load_commons): Use stringpool for binding_label
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-26 Bernhard Reutner-Fischer
* class.c (finalize_component): Use stringpool.
(finalization_scalarizer): Likewise.
* frontend-passes.c (create_var): Likewise.
(get_len_trim_call): Likewise
From: Bernhard Reutner-Fischer
gfc_match_name does check for too long names already. Since
gfc_new_symbol is also called for symbols with internal names containing
compiler-generated prefixes, these internal names can easily exceed the
max_identifier_length mandated by the standard.
gcc/fortran
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-29 Bernhard Reutner-Fischer
* module.c (load_needed): Use stringpool when generating symbols
and module names.
---
gcc/fortran/module.c | 17 -
1 file changed, 12 insertions(+), 5 deletions
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-12-12 Bernhard Reutner-Fischer
* trans-openmp.c (gfc_trans_omp_array_reduction_or_udr): Use
stringpool for clause reduction code.
---
gcc/fortran/trans-openmp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a
From: Bernhard Reutner-Fischer
compiling gfortran.dg/typebound_proc_31.f90 leaked the type-bound
structs:
56 bytes in 1 blocks are definitely lost.
at 0x4C2CC05: calloc (vg_replace_malloc.c:711)
by 0x151EA90: xcalloc (xmalloc.c:162)
by 0x8E3E4F: gfc_get_typebound_proc(gfc_typebound_proc
From: Bernhard Reutner-Fischer
This replaces the remaining occurrences of names and name manipulation
to go through the stringpool. Required to make TYPE (IS) handling work
later on.
gcc/fortran/ChangeLog:
2017-11-14 Bernhard Reutner-Fischer
* class.c (gfc_build_class_symbol): Use
From: Bernhard Reutner-Fischer
Use the existing helper function to create type names. The helper
function uses the stringpool already.
gcc/fortran/ChangeLog:
2017-11-24 Bernhard Reutner-Fischer
* decl.c (build_sym): Use stringpool node instead of stack
variables
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-24 Bernhard Reutner-Fischer
* primary.c (match_charkind_name): Return stringpool node.
(match_string_constant): Use stringpool node for name.
---
gcc/fortran/primary.c | 21 -
1 file changed
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-24 Bernhard Reutner-Fischer
* class.c (finalizer_insert_packed_call): Use stringpool.
(generate_finalization_wrapper): Likewise.
(gfc_find_derived_vtab): Likewise.
(find_intrinsic_vtab): Likewise
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-23 Bernhard Reutner-Fischer
* module.c (use_iso_fortran_env_module): Use stringpool and use
pointer comparison instead of strcmp.
---
gcc/fortran/module.c | 16
1 file changed, 8 insertions(+), 8
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-20 Bernhard Reutner-Fischer
* gfortran.h (struct CInteropKind_t): Make name a pointer.
* misc.c (get_c_kind): Use pointer comparison on name to
determine index.
* symbol.c
From: Bernhard Reutner-Fischer
gcc/fortran/ChangeLog:
2017-11-23 Bernhard Reutner-Fischer
* module.c (import_iso_c_binding_module): Use pointer comparison
instead instead of strcmp.
---
gcc/fortran/module.c | 25 ++---
1 file changed, 10 insertions
On Tue, 21 Jun 2016 at 00:19, Jeff Law wrote:
>
> On 06/18/2016 01:31 PM, Bernhard Reutner-Fischer wrote:
> > A branch with a name matching scan-assembler pattern triggers
> > inappropriate FAIL.
> >
> > E.g. branch fixups-testsuite and
> > - gcc.target/i386/p
On 5 September 2018 20:44:05 CEST, Janne Blomqvist
wrote:
>On Wed, Sep 5, 2018 at 6:00 PM Bernhard Reutner-Fischer <
>rep.dot@gmail.com> wrote:
>
>> diff --git a/gcc/fortran/match.c b/gcc/fortran/match.c
>> index 38827ed4637..6596bd87c09 100644
>> --- a
On Wed, 5 Sep 2018 at 20:57, Janne Blomqvist wrote:
>
> On Wed, Sep 5, 2018 at 5:58 PM Bernhard Reutner-Fischer
> wrote:
>> Bootstrapped and regtested on x86_64-foo-linux.
>>
>> I'd appreciate if someone could double check for regressions on other
>> setu
Janne,
On Mon, 27 Nov 2017 at 20:57, Thomas Koenig wrote:
>
> Hi Janne,
>
> > PING
>
> Looks good for trunk. Thanks!
AFAICS this removed the last user of
static void
init_unsigned_integer
which manifests itself as
../../../../../src/gcc-9.0.mine/libgfortran/runtime/environ.c:104:1:
warning: ‘
101 - 200 of 906 matches
Mail list logo
