By this argument there is a missing warning for the equivalent:
const int foo = 123;
with no previous extern declaration.
As well, there is no warning in C++.
All three constructs are equivalent, yet only one gets a warning.
Interesting point, that I had not realized, and with an often
On 21/01/18 08:12, Jay K wrote:
> extern const int foo = 123;
>
>
>
> Why does this warn?
> This is a valid portable form, with the same meaning
> across all compilers, and, importantly, portably
> to C and C++.
>
> I explicitly do not want to say:
>
> const int foo = 123
>
> because I
A second release candidate for GCC 7.3 is available from
ftp://gcc.gnu.org/pub/gcc/snapshots/7.3.0-RC-20180122/
and shortly its mirrors. It has been generated from SVN revision 256937.
I have so far bootstrapped and tested the release candidate on
x86_64-unknown-linux-gnu. Please test
On 22/01/2018 10:31, Jay K wrote:
By this argument there is a missing warning for the equivalent:
const int foo = 123;
with no previous extern declaration.
I would like to see such a warning. There is "-Wmissing-declarations",
but that applies only to functions and not to objects.
I tried this:
struct C {
virtual ~C();
virtual void f();
};
void
f (C *p)
{
p->f();
p->f();
}
with r256939 and -mindirect-branch=thunk -O2 on x86-64 GNU/Linux, and
got this:
_Z1fP1C:
.LFB0:
.cfi_startproc
pushq %rbx
.cfi_def_cfa_offset 16
Hi,
On Fri, Jan 19 2018, Sandra Loosemore wrote:
> On 01/19/2018 10:14 AM, Jeff Law wrote:
>
>> cc0 needs to die. That doesn't mean that any particular target needs to
>> be dropped -- it just means that someone has to step forward to do the
>> conversion.
>
> Unifying two parallel threads:
Am 2018-01-21 um 13:08 schrieb Georg-Johann Lay:
Jay K schrieb:
extern const int foo = 123;
Why does this warn?
This is a valid portable form, with the same meaning
across all compilers, and, importantly, portably
to C and C++.
I also wondered about this.
In C99 §6.9.2 "External object
On Mon, Jan 22, 2018 at 10:57:35AM +0100, Martin Jambor wrote:
> Hi,
>
> On Fri, Jan 19 2018, Sandra Loosemore wrote:
> > On 01/19/2018 10:14 AM, Jeff Law wrote:
> >
> >> cc0 needs to die. That doesn't mean that any particular target needs to
> >> be dropped -- it just means that someone has to
> If you put static (non-const)
> variables in your header files, you have misunderstood how to use header
> files in C programming.
Not me, and usually const, but people do it, both.
Even the consts can get duplicated.
Even the simple C++
const int one = 1;
I can take the address
Also the warning did not include a link explaining the desired workaround.
Since you advocate for static...and I know it has big value..
There are the following reasons against static:
- It is prohibited in some coding conventions.
They instead hide symbols by omitting them from any
Hi,
I made some points in my other reply. But for completeness, I'll tackle
these too.
On 22/01/2018 10:38, Jay K wrote:
Also the warning did not include a link explaining the desired workaround.
Since you advocate for static...and I know it has big value..
There are the following
> I find the "scoping" too hard to pass it, and if I need to make
> the symbol extern in future, I can afford a rename to do it.
I mean, I actually like like the ability to shorten file level static symbols.
As you point out, true, you can have it both ways, static does not force
On 22/01/2018 11:14, Jay K wrote:
I meant:
extern const foo = 123;
does not warn in C++, but by these arguments, should.
Yes, I think it should. But I am a compiler user, not a compiler
author, so my bias is strongly towards /my/ code rather than a wider
audience.
I understand
Hello,
I'll be contributing to GCC by working on OpenCoarrays as my PhD programme's
broadening project. I'd like to request the necessary forms to do so. As far as
I'm aware, those are the copyright assignment form of all future changes and
the employer disclaimer form.
Best wishes,
Daniel
Hi -
> Problems are still occurring for me; Bugzilla gives me 504 Gateway
> Time-outs when I try to access it tonight...
OK, we reworked some of the database routine maintenance workload,
e.g., a nightly cleanup pass that was quite likely excessive, and
will keep monitoring.
- FChE
* Manuel Rigger:
> Details: We downloaded all C projects from GitHub that had more than 80
> GitHub stars, which yielded almost 5,000 projects with a total of more
> than one billion lines of C code. We filtered GCC, forks of GCC, and
> other compilers as we did not want to incorporate internal
Hi,
On Fri, 12 Jan 2018, Joseph Myers wrote:
> On Fri, 12 Jan 2018, Alexander Monakov wrote:
>
> > No. The qsort_chk effort was limited to catching instances where comparators
> > are invalid, i.e. lack anti-commutativity (may indicate A < B < A) or
> > transitivity property (may indicate A < B
Hi,
I think we are getting quite a bit off-topic for the gcc list here. We
should probably take the discussion to somewhere like comp.lang.c. So I
shall limit things to just a couple of points to round off.
On 22/01/2018 12:27, Jay K wrote:
> If you put static (non-const)
> variables
On 21 January 2018 at 07:12, Jay K wrote:
> extern const int foo = 123;
>
>
>
> Why does this warn?
> This is a valid portable form, with the same meaning
> across all compilers, and, importantly, portably
> to C and C++.
>
> I explicitly do not want to say:
>
> const int foo = 123
>
> because I
On 21 January 2018 at 12:08, Georg-Johann Lay wrote:
> Jay K schrieb:
>>
>> extern const int foo = 123;
>>
>> Why does this warn?
>> This is a valid portable form, with the same meaning
>> across all compilers, and, importantly, portably
>> to C and C++.
>
>
> I also wondered about this.
>
> In
Hi!
On Mon, Jan 22, 2018 at 10:57:35AM +0100, Martin Jambor wrote:
> On Fri, Jan 19 2018, Sandra Loosemore wrote:
> > On 01/19/2018 10:14 AM, Jeff Law wrote:
> >
> >> cc0 needs to die. That doesn't mean that any particular target needs to
> >> be dropped -- it just means that someone has to
Sent from my iPhone
On Mon, Jan 22, 2018 at 04:55:42PM +0100, David Brown wrote:
> Many of these are going to be used automatically by the compiler. You
> write "strdup" in your code, and the compiler treats it as
> "__builtin_strdup". I don't know that such functions need to be
> documented as extensions, but they
On Mon, Jan 22, 2018 at 7:55 AM, David Brown wrote:
> On 22/01/18 16:46, Manuel Rigger wrote:
>> Hi everyone,
>>
>> As part of my research, we have been analyzing the usage of GCC builtins
>> in 5,000 C GitHub projects. One of our findings is that many of these
>> builtins
On 2018-01-22 10:53:55 +0100, David Brown wrote:
> On 22/01/2018 10:31, Jay K wrote:
> >
> > By this argument there is a missing warning for the equivalent:
> >
> > const int foo = 123;
> >
> > with no previous extern declaration.
>
> I would like to see such a warning. There is
On 1/22/2018 9:55 AM, David Brown wrote:
On 22/01/18 16:46, Manuel Rigger wrote:
Hi everyone,
As part of my research, we have been analyzing the usage of GCC builtins
in 5,000 C GitHub projects. One of our findings is that many of these
builtins are unused, even though they are described in
On 01/19/2018 03:09 PM, Martin Jambor wrote:
> Hi,
>
> On Thu, Jan 18 2018, Martin Liška wrote:
>> On 01/17/2018 06:54 PM, Martin Jambor wrote:
>>
> ...
>>>
>>> 2) Martin Liška is willing to mentor either:
>>>2a) -fsanitize=type (He provided URL https://reviews.llvm.org/D32197
>>>but
Hi everyone,
As part of my research, we have been analyzing the usage of GCC builtins
in 5,000 C GitHub projects. One of our findings is that many of these
builtins are unused, even though they are described in the documentation
(see
On 22/01/18 16:46, Manuel Rigger wrote:
> Hi everyone,
>
> As part of my research, we have been analyzing the usage of GCC builtins
> in 5,000 C GitHub projects. One of our findings is that many of these
> builtins are unused, even though they are described in the documentation
> (see
On 01/19/2018 03:13 PM, Martin Jambor wrote:
> On Fri, Jan 19 2018, Martin Jambor wrote:
>> Hi,
>>
>> On Thu, Jan 18 2018, Martin Liška wrote:
>>> On 01/17/2018 06:54 PM, Martin Jambor wrote:
>>>
>> ...
2) Martin Liška is willing to mentor either:
2a) -fsanitize=type (He provided
> On Jan 22, 2018, at 5:17 AM, Trevor Saunders wrote:
>
> On Mon, Jan 22, 2018 at 10:57:35AM +0100, Martin Jambor wrote:
>> Hi,
>>
>> On Fri, Jan 19 2018, Sandra Loosemore wrote:
>>> On 01/19/2018 10:14 AM, Jeff Law wrote:
>>>
cc0 needs to die. That doesn't mean
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83950
Richard Biener changed:
What|Removed |Added
Keywords||rejects-valid
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83952
Richard Biener changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83949
--- Comment #5 from Richard Biener ---
Created attachment 43201
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43201=edit
unincluded unreduced testcase
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81360
--- Comment #11 from Jan Hubicka ---
Hmm, this is different issue introduced by Richard's change to reorder can
inline and want inline.
2018-01-14 Richard Sandiford
* ipa-inline.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81360
Martin Liška changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
Currently gcc_release builds GCC (for generating in-tree generated
files) serially - that's prohibitly expensive and takges ages (>4h for
me). I'm using (when I remember to edit gcc_release ...) -j6 without
a problem for some years, thus the following proposal.
Any recommendation for the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83954
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
On Mon, Jan 22, 2018 at 09:37:05AM +0100, Richard Biener wrote:
>
> Currently gcc_release builds GCC (for generating in-tree generated
> files) serially - that's prohibitly expensive and takges ages (>4h for
> me). I'm using (when I remember to edit gcc_release ...) -j6 without
> a problem for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83960
Richard Biener changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83958
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
Version|unknown
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82005
Richard Biener changed:
What|Removed |Added
CC||simon at pushface dot org
--- Comment
Hi,
a mail in the gcc-list https://gcc.gnu.org/ml/gcc/2018-01/msg00144.html
reminded me about this minor stuff I have (so it can be controlled by
-Werror),
but never managed to write testcases. I'm posting it here in case
it's good enough or someone wants to take over from here.
Franz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83946
Richard Biener changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83877
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83966
Richard Biener changed:
What|Removed |Added
Keywords||diagnostic
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83962
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |8.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83892
--- Comment #5 from rguenther at suse dot de ---
On Sun, 21 Jan 2018, simon at pushface dot org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83892
>
> simon at pushface dot org changed:
>
>What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83945
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83947
Richard Biener changed:
What|Removed |Added
Keywords||ice-on-invalid-code
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83949
Richard Biener changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83954
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83952
--- Comment #10 from Richard Biener ---
But at least we could perform strength reduction in IVOPTs, replacing
for (i = 0; i < 100; i++) {
val = 2 * i;
a[i] = val;
}
with
for (i = 0, j = 0; i < 100; i++, j+=2) {
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83878
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83964
Richard Biener changed:
What|Removed |Added
Version|unknown |8.0
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83965
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
Version|unknown
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83963
Richard Biener changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83965
rsandifo at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83967
Bug ID: 83967
Summary: LTO removes C functions declared as weak in
assembler(depending on files order in linking)
Product: gcc
Version: 7.2.1
Status: UNCONFIRMED
Hi,
I tried omp_clause_mask and it looks ok. But it lacks check if there is any bit
or none. With addition of it(as proposed or in some other way it should work.
What do you think about this approach(patch attached)?
Thanks,
Julia
> -Original Message-
> From: Jakub Jelinek
On 01/21/2018 09:21 AM, Ville Voutilainen wrote:
Finishing testing the attached, OK for trunk?
ok (I had thought typedefs would be covered by the decl_context ==
TYPENAME case, but I see that is not true.)
nathan
--
Nathan Sidwell
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83970
Bug ID: 83970
Summary: -mindirect-branch=thunk -fno-plt generates
CET-incompatible thunk
Product: gcc
Version: 8.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83895
--- Comment #2 from ville at gcc dot gnu.org ---
Author: ville
Date: Mon Jan 22 12:44:33 2018
New Revision: 256942
URL: https://gcc.gnu.org/viewcvs?rev=256942=gcc=rev
Log:
PR c++/83895
cp/
* decl.c (grokdeclarator): Don't diagnose extra parens
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81443
--- Comment #14 from Eric Botcazou ---
It's rather a combinatorial explosion than an unbounded recursion.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935
Pierre-Marie de Rodat changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71892
dnahrblock changed:
What|Removed |Added
CC||howunijemu at crypemail dot
info
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=27931
dnahrblock changed:
What|Removed |Added
CC||howunijemu at crypemail dot
info
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83969
Bug ID: 83969
Summary: [8 Regression] ICE in final_scan_insn, at final.c:2997
(error: could not split insn) for powerpc targets
Product: gcc
Version: 8.0
Status:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83879
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80481
Andrew Senkevich changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83958
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
On Mon, Jan 22, 2018 at 11:30:10AM +, Koval, Julia wrote:
> Hi, I tried omp_clause_mask and it looks ok. But it lacks check if there
> is any bit or none. With addition of it(as proposed or in some other way
> it should work. What do you think about this approach(patch attached)?
Well, I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83969
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |8.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83957
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83895
Ville Voutilainen changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Dne 2018-01-19 22:45, Jakub Jelinek napsal:
Hi!
This PR is about a certain test FAILing on arm, because it scans for
"Invalid sum ..." message in the *.ira dump, but recent GCC versions
have
that message in there; not introduced by IRA though, but all the way
from
expansion. We are
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83935
--- Comment #2 from Pierre-Marie de Rodat ---
Thinking more about it, the rule that the discriminant entry must be a child of
the variant part entry sounds suspicious to me.
In the case of two variant parts, which are nested and depend on the
Hi Kyrill
On 19/01/18 18:00, Kyrill Tkachov wrote:
On 16/01/18 10:31, Sudakshina Das wrote:
Hi Christophe
On 12/01/18 18:32, Christophe Lyon wrote:
Le 12 janv. 2018 15:26, "Sudakshina Das" a écrit :
Hi
This patch fixes my earlier test case that fails for arm-none-eabi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61458
--- Comment #11 from Jonathan Wakely ---
That's not an ABI break. Changing it now would be an ABI break.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2018-01-22 Richard Biener
PR tree-optimization/83963
* graphite-scop-detection.c (scop_detection::get_sese): Delay
including the loop exit block.
On 19 January 2018 at 22:45, Jakub Jelinek wrote:
> Hi!
>
> This PR is about a certain test FAILing on arm, because it scans for
> "Invalid sum ..." message in the *.ira dump, but recent GCC versions have
> that message in there; not introduced by IRA though, but all the way
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=34300
dnahrblock changed:
What|Removed |Added
CC||howunijemu at crypemail dot
info
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65018
dnahrblock changed:
What|Removed |Added
CC||howunijemu at crypemail dot
info
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49915
dnahrblock changed:
What|Removed |Added
CC||howunijemu at crypemail dot
info
---
Hi all,
This test has needlessly restrictive requirements. It tries to force a
soft-float target and tries to run.
This makes it unsupportable for any non-soft-float variant.
In fact, the test can be a run-time test for any target, and only the
scan-assembler tests are specific to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83967
Richard Biener changed:
What|Removed |Added
Keywords||lto, wrong-code
Target|
On Mon, Jan 22, 2018 at 01:11:34PM +0100, Christophe Lyon wrote:
> On 19 January 2018 at 22:45, Jakub Jelinek wrote:
> > That IMHO can't work and the Invalid sum message verifies that. If we want
> > the first jump to hit 99times more often than the second one or vice versa,
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83961
--- Comment #3 from Jeffrey Walton ---
(In reply to Jakub Jelinek from comment #2)
> AFAIK AArch64 has multiple configurable sizes of the virtual address space
> and older GCCs like 6 was only supporting one of those sizes, not all of
> them.
>
Hi,
On 20/01/2018 02:59, Paolo Carlini wrote:
Hi again,
On 19/01/2018 23:55, Paolo Carlini wrote:
...Therefore It seems to me that a way to more cleanly solve the bug
would be moving something like || !DECL_NONTRIVIALLY_INITIALIZED_P to
the the above check in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67804
G. Steinmetz changed:
What|Removed |Added
CC||gs...@t-online.de
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83974
Jakub Jelinek changed:
What|Removed |Added
Priority|P3 |P1
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83974
Bug ID: 83974
Summary: [8 Regression] ICE in cxx_eval_constant_expression
Product: gcc
Version: 8.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
This patch has been triggered by Thomas's recent message to the list.
Not only did I start work late relative to stage 3 but debugging took
somewhat longer than anticipated. Therefore, to get this committed
asap, we will have to beg the indulgence of the release managers and
prompt review and/or
r232820 (aka 2c7b2f8860794cc9b9cf5eeea9d7dc109c0de3be) removed the
implementation of class move_computations_dom_walker, but kept the decl.
This patch removes the stray decl.
Successfully bootstrapped on x86_64-pc-linux-gnu.
OK for trunk?
gcc/ChangeLog:
PR tree-optimization/69452
Dne 2018-01-22 19:36, Jakub Jelinek napsal:
On Mon, Jan 22, 2018 at 06:44:17PM +0100, Jan Hubicka wrote:
> + /* Split *THIS (ORIG) probability into 2 probabilities, such that
> + the returned one (FIRST) is *THIS * CPROB and *THIS is
> + adjusted (SECOND) so that FIRST + FIRST.invert
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81616
--- Comment #49 from Jan Hubicka ---
> matrix.c is still needing additional options to get the best out of the Ryzen
> processor. But is better than before (223029 clocks vs 371978 originally),
> but 122677 is achievable with the right options.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83969
--- Comment #3 from Jakub Jelinek ---
Apparently vectorization, from expand dump it looks like TImode is mode of
vector(2) long long int.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83975
--- Comment #1 from G. Steinmetz ---
Configured with --enable-checking=yes :
$ gfortran-8-20180121-chk -c z1.f90
z1.f90:1:0:
subroutine s(x)
internal compiler error: tree check: expected parm_decl, have var_decl in
1 - 100 of 253 matches
Mail list logo