[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501 --- Comment #2 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:31:43 2015 New Revision: 227582 URL: https://gcc.gnu.org/viewcvs?rev=227582=gcc=rev Log: PR c/67501 * c-parser.c (c_parser_oacc_all_clauses,

[PATCH] Fix ICEs with invalid C #pragma omp atomic (PR c/67495)

2015-09-09 Thread Jakub Jelinek
Hi! The C FE apparently relies on c_parser_unary_expression never called if the expression to be parsed is actually a cast expression, otherwise it fails an assertion. c_parser_cast_expression of course guarantees that and during sizeof and __alignof__ parsing too, but c_parser_omp_atomic calls

[AArch64] Fix vcvt_high_f64_f32 and vcvt_figh_f32_f64 intrinsics.

2015-09-09 Thread James Greenhalgh
Hi, This patch clears up some remaining confusion in the vector lane orderings for the two intrinsics mentioned in the title. Bootstrapped on aarch64-none-linux-gnu and regression tested for aarch64_be-none-elf with no issues. OK? Thanks, James --- 2015-09-09 James Greenhalgh

Re: compiling gcc 5.x from the scratch: issues

2015-09-09 Thread Markus Trippelsdorf
On 2015.09.09 at 08:36 +, Michael Mishourovsky wrote: > At my work I would like to have recent gcc installed but i have no > sudo rights to update the current gcc (its 4.4.7, and OS is redhat > linux). > > So I checked out latest version of gcc via svn, and following > guidelines given

RE: [PATCH] Fix PR64078

2015-09-09 Thread Bernd Edlinger
Hi Jeff, On Tue, 8 Sep 2015 13:27:12, Jeff Law wrote: > > On 09/07/2015 07:46 AM, Bernd Edlinger wrote: >> Hi, >> >> On Mon, 7 Sep 2015 12:07:00, Marek Polacek wrote: >>> >>> On Sun, Sep 06, 2015 at 07:21:13PM +0200, Bernd Edlinger wrote: Hi, we observed sporadic failures of the

[Bug target/67378] PowerPC unrecognizable insn (ICE in in extract_insn, at recog.c:2297)

2015-09-09 Thread amodra at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67378 --- Comment #3 from Alan Modra --- Author: amodra Date: Wed Sep 9 05:59:16 2015 New Revision: 227574 URL: https://gcc.gnu.org/viewcvs?rev=227574=gcc=rev Log: Fix PowerPC ICE due to secondary_reload ignoring reload replacements The reason for

[Bug c/67495] #pragma omp atomic ICEs

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67495 --- Comment #3 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:27:15 2015 New Revision: 227580 URL: https://gcc.gnu.org/viewcvs?rev=227580=gcc=rev Log: PR c/67495 * c-parser.c (c_parser_omp_atomic): Use

[Bug rtl-optimization/65932] [5 Regression] Linux-3.10.75 on arm926ej-s does not boot due to wrong code generation

2015-09-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65932 Ramana Radhakrishnan changed: What|Removed |Added CC||ramana at gcc dot gnu.org ---

[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501 --- Comment #1 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:24:48 2015 New Revision: 227578 URL: https://gcc.gnu.org/viewcvs?rev=227578=gcc=rev Log: PR c/67501 * c-parser.c (c_parser_oacc_all_clauses,

Possible patch for pr62242

2015-09-09 Thread Louis Krupp
This was ... interesting. There were a couple of problems that triggered ICEs. This patch fixes the reported file (I made sure this time) and causes no regressions as far as I can tell. Dominique ... merci de votre patience. Louis Index: gcc/fortran/ChangeLog

[Bug c++/67504] ICE with type dependent collapse argument

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67504 --- Comment #1 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:25:53 2015 New Revision: 227579 URL: https://gcc.gnu.org/viewcvs?rev=227579=gcc=rev Log: PR c++/67504 * parser.c (cp_parser_omp_clause_collapse): Test

[Bug c++/67504] ICE with type dependent collapse argument

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67504 --- Comment #2 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:32:28 2015 New Revision: 227583 URL: https://gcc.gnu.org/viewcvs?rev=227583=gcc=rev Log: PR c++/67504 * parser.c (cp_parser_omp_clause_collapse): Test

Re: [RFC, PR target/65105] Use vector instructions for scalar 64bit computations on 32bit target

2015-09-09 Thread Uros Bizjak
On Wed, Sep 9, 2015 at 10:12 AM, Uros Bizjak wrote: > On Tue, Sep 8, 2015 at 5:49 PM, Ilya Enkovich wrote: >> On 21 Aug 10:38, Jeff Law wrote: >>> On 08/21/2015 07:44 AM, Ilya Enkovich wrote: >>> >>Our of curiosity, what does LLVM do here in terms of

[Bug libstdc++/67478] gcc-5.1.0/gcc-5.2.0 undefined symbol std::__once_callable in AIX 6.1

2015-09-09 Thread tgard at opentext dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67478 --- Comment #4 from Torbjörn Gard --- I found -qtls=model option for xlc in AIX: http://www-01.ibm.com/support/knowledgecenter/SSGH2K_13.1.2/com.ibm.xlc131.aix.doc/compiler_ref/opt_tls.html The corresponding option in gcc is -ftls-model=model.

[Bug c/67500] OpenMP ICE with invalid safelen/simdlen/alignment expressions

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67500 --- Comment #1 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:24:03 2015 New Revision: 227577 URL: https://gcc.gnu.org/viewcvs?rev=227577=gcc=rev Log: PR c/67500 * c-parser.c (c_parser_omp_clause_aligned,

[Bug c/67495] #pragma omp atomic ICEs

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67495 --- Comment #2 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:23:11 2015 New Revision: 227576 URL: https://gcc.gnu.org/viewcvs?rev=227576=gcc=rev Log: PR c/67495 * c-parser.c (c_parser_omp_atomic): Use

Re: [gomp4] force global locks for nvptx targets

2015-09-09 Thread Chung-Lin Tang
On 2015/9/9 04:02 AM, Cesar Philippidis wrote: > This patch forces GOACC_LOCK to use locks in global memory regardless if > the lock us for a worker or a gang. We were using a shared memory for > worker locks, but we ran into an issue with that would sporadically > involve deadlocks in worker

Re: [PATCH][RTL-ifcvt] PR rtl-optimization/67465: Do not ifcvt complex blocks if the else block is empty

2015-09-09 Thread Rainer Orth
Kyrill Tkachov writes: > On 08/09/15 10:26, Rainer Orth wrote: >> Hi Kyrill, >> >>> PR rtl-optimization/67481 is a testsuite regression on sparc-solaris that >>> Rainer reported. I haven't tested >>> that this patch fixes that, but I suspect that the root cause is the >>>

[Bug target/67378] PowerPC unrecognizable insn (ICE in in extract_insn, at recog.c:2297)

2015-09-09 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67378 Alan Modra changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/67510] New: Faster code is possible for integer absolute value

2015-09-09 Thread peter at cordes dot ca
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67510 Bug ID: 67510 Summary: Faster code is possible for integer absolute value Product: gcc Version: 5.2.0 Status: UNCONFIRMED Severity: minor Priority: P3

Re: [PATCH 1/3, libgomp] Adjust offload plugin interface for avoiding deadlock on exit

2015-09-09 Thread Chung-Lin Tang
Ping. On 2015/8/27 09:44 PM, Chung-Lin Tang wrote: > We've discovered that, for several of the libgomp plugin interface routines, > if the target specific routine calls exit() (usually upon a fatal condition), > deadlock ensues. We found this using nvptx, but it's possible on intelmic as > well.

Re: [PATCH] Make ubsan tests less picky about ansi escape codes in diagnostics.

2015-09-09 Thread Maxim Ostapenko
On 09/09/15 01:15, Jonathan Roelofs wrote: On 9/4/15 12:20 AM, Yury Gribov wrote: On 09/03/2015 07:45 PM, Jonathan Roelofs wrote: On 9/3/15 10:17 AM, Jakub Jelinek wrote: On Thu, Sep 03, 2015 at 10:15:02AM -0600, Jonathan Roelofs wrote: +kcc, mrs Ping On 8/27/15 4:44 PM, Jonathan

[Bug target/67378] PowerPC unrecognizable insn (ICE in in extract_insn, at recog.c:2297)

2015-09-09 Thread amodra at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67378 --- Comment #4 from Alan Modra --- Author: amodra Date: Wed Sep 9 06:07:14 2015 New Revision: 227575 URL: https://gcc.gnu.org/viewcvs?rev=227575=gcc=rev Log: Fix PowerPC ICE due to secondary_reload ignoring reload replacements The reason for

[Bug c/67500] OpenMP ICE with invalid safelen/simdlen/alignment expressions

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67500 --- Comment #2 from Jakub Jelinek --- Author: jakub Date: Wed Sep 9 07:30:42 2015 New Revision: 227581 URL: https://gcc.gnu.org/viewcvs?rev=227581=gcc=rev Log: PR c/67500 * c-parser.c (c_parser_omp_clause_aligned,

[PATCH] Fix error recovery on invalid OpenMP clauses (PR c/67501)

2015-09-09 Thread Jakub Jelinek
Hi! If some OpenMP clause is parsed including the argument, such that OMP_CLAUSE tree is created, but there is some error afterwards, the C FE fails to remove the clause if it is not allowed for the current construct, which results in ICEs later on (e.g. during clause splitting). While it would

[Bug libstdc++/67503] String cannot be loaded from binary representation

2015-09-09 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67503 --- Comment #11 from Marc Glisse --- (In reply to radventure from comment #9) > I understand the "small string optimization" idea. I agree about allocation > counting. But I don't see space economy, now sizeof(string) is 28 bytes in > 32-bit

[PATCH] Fix thinko in safelen, simdlen and aligned clause parsing (PR c/67500)

2015-09-09 Thread Jakub Jelinek
Hi! The first 3 hunks are quite obvious, of course we have to complain if any of those conditions are non-zero, rather than only when all of them are non-zero (which never happens). The last hunk fixes ICE if undeclared_variable is called during parsing of #pragma omp declare simd clauses, where

[PATCH] Fix ICE with type dependent collapse argument (PR c++/67504)

2015-09-09 Thread Jakub Jelinek
Hi! The following testcase ICEs because the collapse argument is type dependent and has NULL TREE_TYPE, so testing INTEGRAL_TYPE_P on it ICEs. I hope all INTEGER_CSTs have non-NULL type and thus just swapping the conditions is enough, if not, I could add !type_dependent_expression_p (num) test.

Re: [RFC, PR target/65105] Use vector instructions for scalar 64bit computations on 32bit target

2015-09-09 Thread Uros Bizjak
On Tue, Sep 8, 2015 at 5:49 PM, Ilya Enkovich wrote: > On 21 Aug 10:38, Jeff Law wrote: >> On 08/21/2015 07:44 AM, Ilya Enkovich wrote: >> >>Our of curiosity, what does LLVM do here in terms of costing >> >>models? >> > >> >Unfortunately I have no idea where and how LLVM

compiling gcc 5.x from the scratch: issues

2015-09-09 Thread Michael Mishourovsky
Hello, gcc team. At my work I would like to have recent gcc installed but i have no sudo rights to update the current gcc (its 4.4.7, and OS is redhat linux). So I checked out latest version of gcc via svn, and following guidelines given at gcc webpage, tried to download prerequisites,

[Bug c++/67499] c++ template/overload diagnostic compression

2015-09-09 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67499 --- Comment #8 from Manuel López-Ibáñez --- > If we were outputting html, it would be much easier, we could print the > error message and have javascript so that clicking in appropriate places > displays the candidates, the reason for

Re: [AArch64] Fix vcvt_high_f64_f32 and vcvt_figh_f32_f64 intrinsics.

2015-09-09 Thread Alan Lawrence
Hmmm, hang on. I'm not quite sure what the actual issue/bug is here, but is this the same issue as my patch 12 "with BE RTL fix"? (https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01482.html, explanation last at https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02365.html) I pushed this as r227551

[Bug c++/67318] [6 regression] Parsing error when using abbreviated integral type names in template parameter pack declaration

2015-09-09 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67318 Paolo Carlini changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c++/67318] [6 regression] Parsing error when using abbreviated integral type names in template parameter pack declaration

2015-09-09 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67318 --- Comment #3 from Paolo Carlini --- Created attachment 36310 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36310=edit Draft

[Bug c++/67514] New: ICE in omp_add_variable

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67514 Bug ID: 67514 Summary: ICE in omp_add_variable Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++ Assignee:

[Bug middle-end/67517] New: ICE in gimplify_scan_omp_clauses

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67517 Bug ID: 67517 Summary: ICE in gimplify_scan_omp_clauses Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end

[Bug fortran/52332] Internal compiler error in in gfc_get_symbol_decl

2015-09-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52332 --- Comment #3 from Dominique d'Humieres --- Compiling the code attached to comment 0 with recent versions of gfortran gives now the following error elemental subroutine xml_attribute( attr, name, value )

[Bug sanitizer/67515] New: "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread rogero at howzatt dot demon.co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 Bug ID: 67515 Summary: "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list Product: gcc Version: 6.0 Status: UNCONFIRMED

[Bug c++/67499] c++ template/overload diagnostic compression

2015-09-09 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67499 --- Comment #6 from Marc Glisse --- (In reply to Manuel López-Ibáñez from comment #5) > /usr/include/c++/4.8/ostream:548:5: note: template > std::basic_ostream& std::operator<<(std::basic_ostream _Traits>&, const unsigned

[Bug libstdc++/67503] String cannot be loaded from binary representation

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67503 --- Comment #12 from Jonathan Wakely --- (In reply to radventure from comment #6) > Code validity it's great. But what about backward compatibility? We do not retain backwards compatibility with invalid code. Valid code using std::string still

Re: [AArch64] Fix vcvt_high_f64_f32 and vcvt_figh_f32_f64 intrinsics.

2015-09-09 Thread Christophe Lyon
On 9 September 2015 at 10:31, James Greenhalgh wrote: > > Hi, > > This patch clears up some remaining confusion in the vector lane orderings > for the two intrinsics mentioned in the title. > > Bootstrapped on aarch64-none-linux-gnu and regression tested for >

[Bug c++/67514] ICE in omp_add_variable

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67514 Jakub Jelinek changed: What|Removed |Added Keywords||ice-on-valid-code, openmp

[Bug fortran/60110] ICE in gfc_get_symbol_decl, at fortran/trans-decl.c:1266

2015-09-09 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60110 Dominique d'Humieres changed: What|Removed |Added CC||t56xjcu6dh at snkmail dot com

[Bug middle-end/67512] [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 --- Comment #2 from Marek Polacek --- Here, invert_tree_comparison is asked to return the logical inverse for BIT_AND_EXPR and crashes on that.

[Bug sanitizer/67513] New: ASAN: Not optimal shadow value check (unlikely condition checked in fast path)

2015-09-09 Thread ryabinin.a.a at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67513 Bug ID: 67513 Summary: ASAN: Not optimal shadow value check (unlikely condition checked in fast path) Product: gcc Version: 5.2.0 Status: UNCONFIRMED Severity:

[Aarch64][target/PR 67143][5.2] Backport correct constraints for atomic operations.

2015-09-09 Thread Matthew Wahab
The Aarch64 instruction patterns for atomic operations on memory use the same constraints for arithmetic and logical operations despite the arithmetic operations requiring stronger restrictions. This causes an ICE in some circumstances. This patch backports the fix from trunk to the GCC-5

[Bug middle-end/67512] [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 --- Comment #4 from Marek Polacek --- --- a/gcc/tree-ssa-uninit.c +++ b/gcc/tree-ssa-uninit.c @@ -1296,7 +1296,8 @@ pred_equal_p (pred_info x1, pred_info x2) return false; c1 = x1.cond_code; - if (x1.invert != x2.invert) + if

[Bug c++/67511] New: ICE with invalid OpenMP random access iterator

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67511 Bug ID: 67511 Summary: ICE with invalid OpenMP random access iterator Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug middle-end/67512] New: [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 Bug ID: 67512 Summary: [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456 Product: gcc Version: 5.2.0 Status: UNCONFIRMED

[Bug middle-end/67512] [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug middle-end/67512] [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 --- Comment #3 from Marek Polacek --- (as it should, it is only supposed to handle tcc_comparisons)

[Bug target/53987] [SH] Unnecessary zero-extensions

2015-09-09 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53987 --- Comment #13 from Oleg Endo --- attachment 36309 of PR 67506 is another example. It contains the following code sequence: mov.b @r2,r0 << extu.b r0,r0<< tst #128,r0 bt/s.L11 extu.w

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

2015-09-09 Thread David Edelsohn
On Wed, Sep 9, 2015 at 12:12 AM, Ian Lance Taylor wrote: > Mike Stump writes: > >> Not a big issue, but slightly better if (O_CLOEXEC>>32) != 0 is also >> true. See, if AIX should ever define this to a sensible value, the >> above would disappear the

[Bug objc/67516] New: class.c left shift of 1271241028 by 4 places cannot be represented in type 'int'

2015-09-09 Thread zeccav at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67516 Bug ID: 67516 Summary: class.c left shift of 1271241028 by 4 places cannot be represented in type 'int' Product: gcc Version: 5.2.0 Status: UNCONFIRMED

[Bug target/67506] [5/6 Regression][SH]: error: unrecognizable insn when compiling texlive-binaries

2015-09-09 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67506 Oleg Endo changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/67499] c++ template/overload diagnostic compression

2015-09-09 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67499 --- Comment #7 from Manuel López-Ibáñez --- (In reply to Marc Glisse from comment #6) > In the example above, you dropped the > information that it is 'bar' (the second argument) that cannot be converted, > which, for a function with more

[Bug target/67506] [5/6 Regression][SH]: error: unrecognizable insn when compiling texlive-binaries

2015-09-09 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67506 --- Comment #3 from Oleg Endo --- (In reply to Kazumoto Kojima from comment #2) > Created attachment 36309 [details] > reduced test case > > [...] > > Oleg, could you please look at this? Thanks for reducing it. I will have a look.

[Bug c++/67511] ICE with invalid OpenMP random access iterator

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67511 Jakub Jelinek changed: What|Removed |Added Keywords||error-recovery, openmp

Re: [C++ Patch] PR 53184 ("Unnecessary anonymous namespace warnings")

2015-09-09 Thread Jason Merrill
On 08/24/2015 08:55 AM, Paolo Carlini wrote: + if (no_linkage_check (ftype, /*relaxed_p=*/false)) How about using the return value of no_linkage_check in the warning? Jason

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread rogero at howzatt dot demon.co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 --- Comment #4 from Roger Orr --- Ah - apologies -- I'd got the example by stripping down a call in boost::format and didn't do a full enough check that the code was well formed: I'll report that UB to boost. However as Markus says the seg

[hsa] Avoid setting register definitions when adding operands to a phi node

2015-09-09 Thread Martin Jambor
Hi, with the new operand setter method, we've been attempting to set definition of a register when we were adding it to the zeroth operand of a phi node. Fortunately, there is an assert to detect the situation. Fixed thusly, will commit to the branch after rudimentary testing. Thanks, Martin

[Bug sanitizer/67515] crash from ubsan for non-virtual call in initializer list with an invalid vtable

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 --- Comment #8 from Jakub Jelinek --- You can get the same segfault with clang++ e.g. on struct A { int a; A () {} int foo () { return 1; } virtual ~A () {} }; alignas (A) char buf[sizeof (A)]; void foo (void *x) { A *y = (A *) x;

[Bug middle-end/67518] New: [6 Regression] ISL: position out of bounds

2015-09-09 Thread Joost.VandeVondele at mat dot ethz.ch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67518 Bug ID: 67518 Summary: [6 Regression] ISL: position out of bounds Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

2015-09-09 Thread Jonathan Wakely
On 08/09/15 21:52 +0200, François Dumont wrote: Very good approach, I will start moving light checks from _GLIBCXX_DEBUG implementation to normal one then. Great, thanks. I *think* my patch got most of the lightweight checks already. We could maybe include the irreflexivity checks, although

[Bug middle-end/67512] [5/6 Regression] internal compiler error: in invert_tree_comparison, at fold-const.c:2456

2015-09-09 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67512 --- Comment #5 from Marek Polacek --- With the patch we don't ICE anymore but we lose this warning: ice.c:10:3: warning: ‘z’ may be used uninitialized in this function [-Wmaybe-uninitialized] fn3 (z); ^ but I think that's not a fault of

[Bug sanitizer/67258] "invalid vptr" false positive from ubsan for virtual inheritance

2015-09-09 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67258 Markus Trippelsdorf changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[PTX] atomics compare exchange

2015-09-09 Thread Nathan Sidwell
I noticed the atomic compare exchange was using 'and 1' to generate a bool. We now have select, which makes it more obvious. Sadly the middle end's expansion of of the builtin fails to make use of the condition we just generated directly, instead generating a new one from the bool. nathan

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 --- Comment #2 from Jonathan Wakely --- A message about a vptr is a bit mis-leading for non-virtual call, so maybe that could be improved, but in essence 'this' is not well-defined at that point.

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 Markus Trippelsdorf changed: What|Removed |Added Status|RESOLVED|REOPENED Last reconfirmed|

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 --- Comment #6 from Jakub Jelinek --- The problem is that to avoid the segfault, you'd need to significantly slow down the library code (pretty much, instead of if (Prefix->Offset > 0 || !Prefix->TypeInfo) // This can't possibly be a valid

[Bug c++/67519] New warning: calls to member functions before all base classes constructed

2015-09-09 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67519 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug middle-end/67517] ICE in gimplify_scan_omp_clauses

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67517 Jakub Jelinek changed: What|Removed |Added Keywords||openmp Status|UNCONFIRMED

[Bug other/67520] New: libmpx should abort() instead of exit(255) on bound violation detection

2015-09-09 Thread jussi.judin at ericsson dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67520 Bug ID: 67520 Summary: libmpx should abort() instead of exit(255) on bound violation detection Product: gcc Version: 5.2.0 Status: UNCONFIRMED Severity:

[Bug libobjc/67516] class.c left shift of 1271241028 by 4 places cannot be represented in type 'int'

2015-09-09 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67516 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug c/65892] gcc fails to implement N685 aliasing of union members

2015-09-09 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65892 --- Comment #14 from joseph at codesourcery dot com --- That C++ wording doesn't have any obvious bearing on what "it is permitted" is intended to be an exception to - the general implementation-defined nature of type punning (which I think

[PATCH] Fix ICE in predicate comparison in uninit (PR middle-end/67512)

2015-09-09 Thread Marek Polacek
We were crashing on this testcase in invert_tree_comparison because it got BIT_AND_EXPR, but this function expects comparison codes only. In this case pred_equal_p got two predicates: m != 1 and m & 1. By checking the tcc_comparison first we don't ICE anymore and pred_equal_p correctly says

[Bug c/65892] gcc fails to implement N685 aliasing of union members

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65892 --- Comment #15 from Jonathan Wakely --- (In reply to Melissa from comment #12) > A C++ conversion of the original example is below. I asked about the word > "read" on the C++ Standard Discussion (std-discussion) mailing list, because > it

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug sanitizer/67515] crash from ubsan for non-virtual call in initializer list with an invalid vtable

2015-09-09 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 Andrew Pinski changed: What|Removed |Added Summary|"invalid vptr" false|crash from ubsan for

[Bug c++/67519] New: New warning: calls to member functions before all base classes constructed

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67519 Bug ID: 67519 Summary: New warning: calls to member functions before all base classes constructed Product: gcc Version: 4.9.4 Status: UNCONFIRMED Keywords:

[Bug sanitizer/67515] "invalid vptr" false positive or crash from ubsan for non-virtual call in initializer list

2015-09-09 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67515 --- Comment #5 from Markus Trippelsdorf --- (anonymous namespace)::getVtablePrefix (Object=0x401460 ) at ../../../../gcc/libsanitizer/ubsan/ubsan_type_hash.cc:200 200 if (Prefix->Offset > 0 || !Prefix->TypeInfo) (gdb) bt #0 (anonymous

Re: [PATCH, fortran] PR 53379 Backtrace on error termination

2015-09-09 Thread Mike Stump
On Sep 8, 2015, at 9:12 PM, Ian Lance Taylor wrote: > Yes, I think this might be even better in code. How about something > like > > /* On some versions of AIX O_CLOEXEC does not fit in int, so use a > cast to force it. */ > descriptor = open (filename, (int) (O_RDONLY |

Re: [PATCH] Make ubsan tests less picky about ansi escape codes in diagnostics.

2015-09-09 Thread Andrew Pinski
On Wed, Sep 9, 2015 at 11:11 PM, Mike Stump wrote: > On Sep 3, 2015, at 9:45 AM, Jonathan Roelofs > wrote: >> Moral of the story is: these tests fail in our environment, but only because >> the regexes do not expect the presence of the ansi

[Bug target/67366] Poor assembly generation for unaligned memory accesses on ARM v6 & v7 cpus

2015-09-09 Thread ramana at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67366 --- Comment #12 from Ramana Radhakrishnan --- (In reply to rguent...@suse.de from comment #3) > On Thu, 27 Aug 2015, rearnsha at gcc dot gnu.org wrote: > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67366 > > > > --- Comment #2 from

Re: [PATCH] [MIPS] Fix wrong instruction in the delay slot

2015-09-09 Thread Jeff Law
On 09/04/2015 09:19 AM, Robert Suchanek wrote: It appears that a possibly related issue is already reported to Bugzilla (bug 51513) where the branch is not optimized away, leaving the compare and branch instructions. It would also appear that this should be fixed at the tree level, however,

Advertisement in the GCC mirrors list

2015-09-09 Thread niXman
Hi, http://mirrors-ru.go-parts.com/gcc - Online Shop ftp://mirrors-ru.go-parts.com/gcc - bad rsync://mirrors-ru.go-parts.com/gcc - bad http://mirrors-uk.go-parts.com/gcc/ - Online Shop ftp://mirrors-uk.go-parts.com/gcc - bad rsync://mirrors-uk.go-parts.com/gcc - bad

[Bug middle-end/67521] New: ICE when OpenMP loop expressions mention the IV

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67521 Bug ID: 67521 Summary: ICE when OpenMP loop expressions mention the IV Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug middle-end/67521] ICE when OpenMP loop expressions mention the IV

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67521 Jakub Jelinek changed: What|Removed |Added Keywords||openmp Status|UNCONFIRMED

Re: [PATCH] Make ubsan tests less picky about ansi escape codes in diagnostics.

2015-09-09 Thread Mike Stump
On Sep 3, 2015, at 9:45 AM, Jonathan Roelofs wrote: > Moral of the story is: these tests fail in our environment, but only because > the regexes do not expect the presence of the ansi color codes, and we can't > trick the runtime into not emitting them. When the user

Re: [PATCH] Fix ICE in predicate comparison in uninit (PR middle-end/67512)

2015-09-09 Thread Jeff Law
On 09/09/2015 08:02 AM, Marek Polacek wrote: We were crashing on this testcase in invert_tree_comparison because it got BIT_AND_EXPR, but this function expects comparison codes only. In this case pred_equal_p got two predicates: m != 1 and m & 1. By checking the tcc_comparison first we don't

Re: [PATCH] Convert SPARC to LRA

2015-09-09 Thread Mike Stump
On Sep 8, 2015, at 9:41 PM, David Miller wrote: > +#define TARGET_LRA_P hook_bool_void_true Are we at the point there this should be the default, and old ports should just define to false, if they really need to? I’m using nothing but LRA as well.

Re: [PATCH] Prevent unnecessary recompilation for trivial params.def changes

2015-09-09 Thread Tom de Vries
On 08/09/15 23:08, Jeff Law wrote: On 09/08/2015 07:21 AM, Tom de Vries wrote: [ was: Re: [RFC] Prevent unnecessary recompilation for trivial params.def changes ] On 08/09/15 14:03, Andreas Schwab wrote: Tom de Vries writes: After a subsequent rebuild I don't see

Re: [PATCH] Fix PR64078

2015-09-09 Thread Jeff Law
On 09/09/2015 03:10 AM, Bernd Edlinger wrote: Hi Jeff, On Tue, 8 Sep 2015 13:27:12, Jeff Law wrote: On 09/07/2015 07:46 AM, Bernd Edlinger wrote: Hi, On Mon, 7 Sep 2015 12:07:00, Marek Polacek wrote: On Sun, Sep 06, 2015 at 07:21:13PM +0200, Bernd Edlinger wrote: Hi, we observed

[Bug c/65892] gcc fails to implement N685 aliasing of union members

2015-09-09 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65892 --- Comment #16 from Jonathan Wakely --- Message [c++std-core-20893] on the C++ core reflector on 2011-12-14 supports the GCC view that a C++ compiler can apply strict aliasing rules to p1->m and p2->m unless the fact they come from the same

[Bug c++/67522] New: OpenMP ICE in type_dependent_expression_p

2015-09-09 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67522 Bug ID: 67522 Summary: OpenMP ICE in type_dependent_expression_p Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++

Fwd: Successful GCC 5.2 installation

2015-09-09 Thread Anirudh Attri
Hi, The GCC 5.2 installation was successful on an HP Pavilion x360. The outputs required are: Output of config.guess: x86_64-unknown-linux-gnu Output of gcc -v: Using built-in specs. COLLECT_GCC=gcc COLLECT_LTO_WRAPPER=/usr/local/libexec/gcc/x86_64-unknown-linux-gnu/5.2.0/lto-wrapper Target:

[PATCH] Fix sanitizer/67258 (by cherry picking upstream patch)

2015-09-09 Thread Markus Trippelsdorf
Tested on ppc64le. OK for trunk and gcc-5? PR sanitizer/67258 * ubsan/ubsan_type_hash.cc: Cherry pick upstream r244101. Upstream patch: commit 1d2477faafda9ad2cc19927b3c31efd22747f013 Author: Alexey Samsonov Date: Wed Aug 5 19:35:46 2015 + [UBSan]

Re: [C++ Patch] PR 53184 ("Unnecessary anonymous namespace warnings")

2015-09-09 Thread Jason Merrill
OK, thanks. Jason

Re: [PATCH] Fix sanitizer/67258 (by cherry picking upstream patch)

2015-09-09 Thread Jakub Jelinek
On Wed, Sep 09, 2015 at 06:18:25PM +0200, Markus Trippelsdorf wrote: > Tested on ppc64le. > OK for trunk and gcc-5? > > PR sanitizer/67258 > * ubsan/ubsan_type_hash.cc: Cherry pick upstream r244101. Please add -fno-sanitize-recover=vptr to dg-options. Ok with that change. > ---

Re: [C++ Patch] PR 53184 ("Unnecessary anonymous namespace warnings")

2015-09-09 Thread Paolo Carlini
Hi, On 09/09/2015 03:00 PM, Jason Merrill wrote: On 08/24/2015 08:55 AM, Paolo Carlini wrote: +if (no_linkage_check (ftype, /*relaxed_p=*/false)) How about using the return value of no_linkage_check in the warning? Agreed, more informative. The below, which I'm finishing testing, also

  1   2   3   >