http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36328
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26190
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26190
--- Comment #3 from Jim Wilson wilson at gcc dot gnu.org ---
This ior/shift/xor optimization be done during combine with a simplify-rtx.c
patch. I wrote a prototype and tested it. Combine canonicalizes shift/logical
as logical/shift, so we
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65664
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66212
--- Comment #2 from Jim Wilson wilson at gcc dot gnu.org ---
libgcc should be built with debug info by default, but the one in /lib is
probably stripped. Try setting LD_LIBRARY_PATH to point at the gcc-5.1
libraries that you built, instead
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66258
--- Comment #2 from Jim Wilson wilson at gcc dot gnu.org ---
Author: wilson
Date: Wed Jun 3 00:46:19 2015
New Revision: 224054
URL: https://gcc.gnu.org/viewcvs?rev=224054root=gccview=rev
Log:
gcc/
PR target/66258
* config
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66258
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66258
--- Comment #3 from Jim Wilson wilson at gcc dot gnu.org ---
The patch doesn't completely fix grub, because grub is trying to compile FP
code with +nofp, which can't work without a soft-float ABI, which we don't
have. So grub gets an error
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66271
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65375
--- Comment #10 from Jim Wilson wilson at gcc dot gnu.org ---
Improved, but not completely resolved. We still get unnecessary orr
instructions, same as in comment 2. This is partly an issue with the register
allocator not handling partially
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #17 from Jim Wilson wilson at gcc dot gnu.org ---
I've been researching how the ARM PROMOTE_MODE reached its current state.
There are a number of issues here.
1) There is a comment that says zero extension is faster for char than
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #18 from Jim Wilson wilson at gcc dot gnu.org ---
Ultimately, I believe that this is an ARM backend bug. PROMOTE_MODE and
TARGET_PROMOTE_FUNCTION_MODE should not behave differently. It would help if
the PROMOTE_MODE macro was merged
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66258
--- Comment #9 from Jim Wilson wilson at gcc dot gnu.org ---
Author: wilson
Date: Tue Jun 16 20:11:41 2015
New Revision: 224538
URL: https://gcc.gnu.org/viewcvs?rev=224538root=gccview=rev
Log:
gcc/
Backport from mainline
2015-06
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #14 from Jim Wilson wilson at gcc dot gnu.org ---
Created attachment 35775
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=35775action=edit
A possibly better patch, to modify ARM port to stop changing signed HI/QI to
unsigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66203
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66271
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #12 from Jim Wilson wilson at gcc dot gnu.org ---
Part of the problem here is that the arm port defines PROMOTE_MODE to force
short local variables to unsigned, but arm_promote_function_mode doesn't change
signedness. So a signed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #13 from Jim Wilson wilson at gcc dot gnu.org ---
Created attachment 35774
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=35774action=edit
Experimental patch to force out-of-ssa to use promoted subregs for phi nodes.
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
palantir:2187$ more tmp.c
#include stdarg.h
int
sub (int num, ...)
{
va_list ap;
int j;
va_start (ap, num);
j = va_arg (ap, int
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66849
Jim Wilson wilson at gcc dot gnu.org changed:
What|Removed |Added
CC||wilson at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66068
--- Comment #8 from Jim Wilson ---
Author: wilson
Date: Wed Oct 28 20:02:01 2015
New Revision: 229505
URL: https://gcc.gnu.org/viewcvs?rev=229505=gcc=rev
Log:
Fix for ICE with -g on testcase with incomplete types.
gcc/c/
PR debug/66068
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66068
Jim Wilson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66785
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66785
--- Comment #3 from Jim Wilson ---
Created attachment 36705
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36705=edit
untested patch that may be wasting memory
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66271
Jim Wilson changed:
What|Removed |Added
Resolution|FIXED |DUPLICATE
--- Comment #9 from Jim Wilson
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932
--- Comment #20 from Jim Wilson ---
*** Bug 66271 has been marked as a duplicate of this bug. ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67305
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66068
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #9 from Jim Wilson ---
(In reply to Andrew Pinski from comment #8)
> (In reply to Jim Wilson from comment #7)
> > The simplified testcases fail on arm if you use -O3 -mfpu=neon.
> >
> > I can look at fixing the arm side of things if
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #11 from Jim Wilson ---
(In reply to Jim Wilson from comment #10)
> Both the aarch64 and arm code looks funny to me, as the last add seems to be
> using an input register that was never set, but I don't know the aarch64 and
> arm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #10 from Jim Wilson ---
(In reply to Jim Wilson from comment #9)
> (In reply to Andrew Pinski from comment #8)
> > (In reply to Jim Wilson from comment #7)
> > > The simplified testcases fail on arm if you use -O3 -mfpu=neon.
> > >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #7 from Jim Wilson ---
The simplified testcases fail on arm if you use -O3 -mfpu=neon.
I can look at fixing the arm side of things if we need an md patch.
: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
Created attachment 37345
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37345=edit
aarch64 testcase generated with creduce, ICE at -O3
I'm seeing an
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #1 from Jim Wilson ---
With an aarch64-linux-gnu cross compiler:
palantir:2654$ ./xgcc -B./ -O3 -S tmp.c
tmp.c: In function 'fn1':
tmp.c:3:5: error: incorrect type of vector CONSTRUCTOR elements
int fn1(void) {
^~~
{_38,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69282
--- Comment #14 from Jim Wilson ---
Andrew Pinksi wrote the patch to fix the ICE. I was expecting him to submit
it.
He also pointed out that the code with the patch is inefficient, and can be
optimized by adding some patterns to the aarch64
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
This is a continuation of bug 69282, which reported an ICE on the same loop,
which has since been fixed. There is still the problem that the code is poorly
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69848
--- Comment #1 from Jim Wilson ---
Created attachment 37716
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37716=edit
code generated by -O2 -ftree-vectorize
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69848
--- Comment #2 from Jim Wilson ---
Created attachment 37717
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37717=edit
better code from hand optimizing the gcc output
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69848
--- Comment #5 from Jim Wilson ---
Created attachment 37737
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37737=edit
Patch to add missing vcond_mask* patterns.
Tested with the subset of CPU2006 that currently works at -O3 on aarch64 and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70422
--- Comment #7 from Jim Wilson ---
The function alloc_entries calls fancy_abort with __func__, causing the string
to get into the output. This function is inlined into the hash_table expand
function, which is then in turn inlined into other
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70519
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #11
++
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
In docs/invoke.texi, it says that 8 is the highest value for -fabi-version. In
c-family/c-opts.c, flag_abi_version gets set to 9. I see two places that check
for abi_version
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
--- Comment #1 from Jim Wilson ---
I see that the patch for bug 65945 was back ported to the gcc-5 branch, which
required a partial backport of the patch for bug 44282, which added abi version
9. The original patch for 44282 is missing the doc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70422
--- Comment #4 from Jim Wilson ---
The broken targets all define flag_section_anchors at -O1 and up. x86_64 does
not. I don't know why this makes a difference yet.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70422
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
--- Comment #2 from Jim Wilson ---
Patch posted for review here
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00480.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
--- Comment #3 from Jim Wilson ---
Patch posted for review here
https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00480.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
--- Comment #4 from Jim Wilson ---
Author: wilson
Date: Tue May 17 18:42:16 2016
New Revision: 236339
URL: https://gcc.gnu.org/viewcvs?rev=236339=gcc=rev
Log:
Make -fabi-version docs match the implementation.
gcc/
PR c++/70613
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
Jim Wilson changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70613
Jim Wilson changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69848
--- Comment #6 from Jim Wilson ---
Testing the vcond_mask* patch with make check gave 6 regressions for both armhf
and aarch64.
FAIL: gcc.dg/vect/pr65947-10.c (internal compiler error)
FAIL: gcc.dg/vect/pr65947-10.c (test for excess errors)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951
--- Comment #7 from Jim Wilson ---
Created attachment 38396
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38396=edit
patch to add a mulv2di3 pattern
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951
--- Comment #10 from Jim Wilson ---
Created attachment 38399
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38399=edit
testcase unsigned long multiply by constant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951
--- Comment #8 from Jim Wilson ---
Created attachment 38397
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38397=edit
patch to use synth_mult to generate vector multiplies
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951
--- Comment #9 from Jim Wilson ---
Created attachment 38398
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38398=edit
testcase signed long multiply by constant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951
--- Comment #11 from Jim Wilson ---
I've spent some time looking at solutions to this problem.
One way to solve it is to simply add a mulv2di3 pattern to the aarch64 port.
The presence of a multiply pattern means we will go through
: normal
Priority: P3
Component: preprocessor
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
In the doc section Environment Variables Affecting GCC
https://gcc.gnu.org/onlinedocs/gcc-6.1.0/gcc/Environment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69848
--- Comment #13 from Jim Wilson ---
I think it was poc_ref_pic_reorder() in slice.c that triggered the ICE. I
don't know if the original code shows the vectorization reduction problem.
That might only be present in the reduced testcase.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71685
--- Comment #4 from Jim Wilson ---
The problem here is that I assumed that c_build_qualified_type would only be
called when we want to create a variant type. However, there are a number of
places that call it without checking to see if we have
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71685
--- Comment #7 from Jim Wilson ---
An early exit if type_quals is 0 does not work, as there is at least one place
that deliberately calls c_build_qualified_type with no type qualifiers to
create an unqualified type from a qualified type.
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
I noticed an arm port scheduler problem. I believe it was introduced by
Richard Earnshaw's patch to add the arm-cpus.in file. There are also some
apparent latent problems exposed
||wilson at gcc dot gnu.org
Resolution|--- |FIXED
--- Comment #5 from Jim Wilson ---
A patch was checked in to fix this about 2.5 years ago.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78064
--- Comment #2 from Jim Wilson ---
The old configure.ac before the patch breaks this has
GCC_TARGET_TEMPLATE(HAVE_GETIPINFO)
which prevents the ifndef/endif from being added to config.in. But this
apparently only works in the gcc dir, and I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78064
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78348
--- Comment #2 from Jim Wilson ---
The testcase doesn't produce runable code, and I'm not sure if I have access to
any haswell parts, but I can make a few comments.
The testcase requires -O3 -ftree-loop-distribution to reproduce.
Without my
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78772
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78731
--- Comment #2 from Jim Wilson ---
The compiler is failing in jump threading as called by dom2. Jeff's 2015-09-30
patch is triggering in dom1, and changing the CFG just enough to prevent the
jump threading bug from triggering. So it isn't a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78529
--- Comment #11 from Jim Wilson ---
FYI I'm using the gdb simulator to run the testcases.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78529
--- Comment #17 from Jim Wilson ---
I still haven't been able to reproduce this, but I do see a problem.
In the original bug report, the only difference is that the code uses x4 in the
first part of the diff, and x24 in the second part of the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78529
--- Comment #14 from Jim Wilson ---
I debugged one of my gcc.c-torture/execute/builtins failures, and found a bug
in the gdb aarch64 simulator with the FP stur instruction support. I submitted
a patch to gdb-patches. With that fix, all of the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78529
--- Comment #26 from Jim Wilson ---
I can reproduce the problem with this new reduced testcase. I don't have
knowledge of all of the details how the gcc implementation of LTO works, but my
understanding goes something like this.
The testcase
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
Creating a bug from the original message, which is here
https://gcc.gnu.org/ml/gcc-bugs/2016-12/msg00678.html
Hi
The following
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78731
Jim Wilson changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78772
--- Comment #11 from Jim Wilson ---
This language has been in the standard since ANSI C 89/ISO C 90.
>extern char buf[32];
>int *a = (int *)&(buf[16]);
>int result = *a; //is this ok? or we have to use memcpy(, [16],
>>sizeof(int))?
This
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
I noticed this while looking at Maxim's testcase in bug 18438 in comment 9,
which is extracted from SPEC CPU2006 462
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
--- Comment #1 from Jim Wilson ---
Patch posted here
https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00058.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81434
--- Comment #1 from Jim Wilson ---
Created attachment 41752
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41752=edit
Proposed patch to fix scheduler/fusing problem.
Priority: P3
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
Consider this testcase extracted from OpenSSL sources
extern int Te0[256];
extern int Te1[256];
extern int Te2[256];
extern int Te3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81434
--- Comment #3 from Jim Wilson ---
Created attachment 41754
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41754=edit
Assembly output with patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81434
--- Comment #2 from Jim Wilson ---
Created attachment 41753
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41753=edit
Assembly output without patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81521
--- Comment #4 from Jim Wilson ---
Author: wilson
Date: Tue Jul 25 16:06:37 2017
New Revision: 250529
URL: https://gcc.gnu.org/viewcvs?rev=250529=gcc=rev
Log:
Fix i686-pc-cygwin build failure.
gcc/
PR bootstrap/81521
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81521
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=2
--- Comment #7 from Jim Wilson ---
I must have failed to realize that this was a bug I had created when I got the
earlier message.
Looking now, I see that I can still reproduce with mainline, though the problem
is not as obvious with this
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78387
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78387
--- Comment #10 from Jim Wilson ---
newunit_stack needs to be allocated dynamically, just like the newunits array.
I modified libgfortran to compute the highest value of newunit_tos, and print
it when the program exits. I set
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81241
Jim Wilson changed:
What|Removed |Added
CC||wilson at gcc dot gnu.org
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
--- Comment #2 from Jim Wilson ---
Author: wilson
Date: Sat May 13 01:32:40 2017
New Revision: 248004
URL: https://gcc.gnu.org/viewcvs?rev=248004=gcc=rev
Log:
Patch for RTL expand bug affecting aarch64 vector code.
gcc/
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
--- Comment #3 from Jim Wilson ---
Needs a testcase for the testsuite.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
--- Comment #4 from Jim Wilson ---
Author: wilson
Date: Fri Jun 23 21:01:32 2017
New Revision: 249612
URL: https://gcc.gnu.org/viewcvs?rev=249612=gcc=rev
Log:
Testcase for FSF GCC bug 79794.
gcc/testsuite/
PR middle-end/79794
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #1 from Jim Wilson ---
Created attachment 41623
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41623=edit
patch to make wrf_s work
Component: libfortran
Assignee: unassigned at gcc dot gnu.org
Reporter: wilson at gcc dot gnu.org
Target Milestone: ---
I'm seeing many different kinds of failures when running a wrf_s binary
compiled with gcc mainline. Double free aborts. Segfaults. Fortran runtime
error: End
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #4 from Jim Wilson ---
Suppose we move the locking inside the if statement. Suppose newunit_tos is 1.
Two threads hit the statement
if (newunit_tos)
at the same time, and both enter the if block. We then hit the lock. The
first
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Jim Wilson changed:
What|Removed |Added
Version|8.0 |7.1.1
--- Comment #7 from Jim Wilson ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #6 from Jim Wilson ---
I should point out that there are currently four places that use
newunit_stack/newunit_tos without locking. Two of these places required
locking code to get wrf working.
The other two are init_units and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #10 from Jim Wilson ---
Author: wilson
Date: Mon Jun 26 21:44:50 2017
New Revision: 249668
URL: https://gcc.gnu.org/viewcvs?rev=249668=gcc=rev
Log:
Fix for SPEC CPu2017 621.wrf_s failure, add missing locking code.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Jim Wilson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #9 from Jim Wilson ---
Author: wilson
Date: Mon Jun 26 21:40:47 2017
New Revision: 249667
URL: https://gcc.gnu.org/viewcvs?rev=249667=gcc=rev
Log:
Fix for SPEC CPU2017 621.wrf_s failure, add missing locking code.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
--- Comment #6 from Jim Wilson ---
Fixed. Patch and testcase committed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79794
Jim Wilson changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
1 - 100 of 402 matches
Mail list logo