How to deal with the different result when executing UB programs?

2020-05-21 Thread Haoxin Tu via Gcc
Hi, there! I am new for using GCC mail list, please forgive me if something is wrong. I have some issues about how GCC deal with the different optimizations in a UB program. For example, small.cc *#include unsigned long long a;void b(unsigned long long *c, int h) { *c = h; }int d =

[Bug rtl-optimization/95267] [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 --- Comment #5 from Andrew Pinski --- (In reply to otcmaf from comment #4) > Do you mean that those pattern above are also wrong pattern ? YES those are broken.

[Bug rtl-optimization/95267] [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread xuemaosheng at huawei dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 --- Comment #4 from otcmaf --- (In reply to Andrew Pinski from comment #3) > The internals documentation documents this even, read: > https://gcc.gnu.org/onlinedocs/gccint/RTL-Template.html#index-match_005fdup > > From that: > Note that

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jakub Jelinek via Gcc
On Thu, May 21, 2020 at 03:12:21PM -0700, Ian Lance Taylor via Gcc wrote: > Hi, this unfortunately breaks gccgo development. Significant parts of > the gccgo sources are simply copied from other repositories. Those > other repositories do not use ChangeLog files. The git commit hook > should

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jakub Jelinek via Gcc-patches
On Thu, May 21, 2020 at 03:12:21PM -0700, Ian Lance Taylor via Gcc wrote: > Hi, this unfortunately breaks gccgo development. Significant parts of > the gccgo sources are simply copied from other repositories. Those > other repositories do not use ChangeLog files. The git commit hook > should

Re: [PATCH][PR92658] Add missing vector truncmn2 expanders for avx512f

2020-05-21 Thread Hongtao Liu via Gcc-patches
On Thu, May 21, 2020 at 7:18 PM Uros Bizjak wrote: > > On Thu, May 21, 2020 at 7:35 AM Hongtao Liu wrote: > > > > On Wed, May 20, 2020 at 11:43 PM Uros Bizjak wrote: > > > > > > On Wed, May 20, 2020 at 10:35 AM Hongtao Liu wrote: > > > > > > > > Hi: > > > > Bootstrap is ok, regression test

[Bug target/92729] [avr] Convert the backend to MODE_CC so it can be kept in future releases

2020-05-21 Thread f.mach4 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729 Max changed: What|Removed |Added CC||f.mach4 at gmail dot com --- Comment #2 from Max

[PATCH v3] Add -fuse-ld= to specify an arbitrary executable as the linker

2020-05-21 Thread Fangrui Song via Gcc-patches
On 2020-05-21, Martin Liška wrote: On 5/21/20 1:52 AM, Fangrui Song wrote: The above issues motivated me to touch this line in PATCH v2. Dropped in PATCH v2. Thank you for the updated patch. The patch is fine except coding style issues: $ ./contrib/check_GNU_style.py

[Bug rtl-optimization/95267] [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 Andrew Pinski changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[Bug testsuite/95110] new test case in r11-345 error: gcc.dg/tree-ssa/pr94969.c: dump file does not exist

2020-05-21 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95110 --- Comment #3 from CVS Commits --- The releases/gcc-9 branch has been updated by Bin Cheng : https://gcc.gnu.org/g:466ad887c9e1cd5a6762e7ec620eef2c8175b50d commit r9-8614-g466ad887c9e1cd5a6762e7ec620eef2c8175b50d Author: Bin Cheng Date:

[Bug tree-optimization/94969] [8/9/10/11 Regression] Invalid loop distribution since r8-2390-gdfbddbeb1ca912c9

2020-05-21 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94969 --- Comment #12 from CVS Commits --- The releases/gcc-9 branch has been updated by Bin Cheng : https://gcc.gnu.org/g:466ad887c9e1cd5a6762e7ec620eef2c8175b50d commit r9-8614-g466ad887c9e1cd5a6762e7ec620eef2c8175b50d Author: Bin Cheng Date:

[Bug rtl-optimization/95267] [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread xuemaosheng at huawei dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 --- Comment #2 from otcmaf --- (In reply to Andrew Pinski from comment #1) > I think this is a back-end issue. > Can you provide the definition of movtv8hf16 ? > I don't think you can do: > (set (match_operand 0 predicate constraint) >

[Bug c++/95221] g++.dg/ubsan/vptr-12.C fails with -fstrong-eval-order=all

2020-05-21 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95221 Jason Merrill changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jason at gcc dot gnu.org

[Bug rtl-optimization/95267] [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Bug rtl-optimization/95267] New: [ICE][gcse]: in process_insert_insn at gcse.c

2020-05-21 Thread xuemaosheng at huawei dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95267 Bug ID: 95267 Summary: [ICE][gcse]: in process_insert_insn at gcse.c Product: gcc Version: 7.3.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [PATCH 1/2] rs6000: tune cunroll for simple loops at O2

2020-05-21 Thread Jiufu Guo via Gcc-patches
Jiufu Guo writes: > Jan Hubicka writes: > >>> Segher Boessenkool writes: >>> >>> > On Wed, May 20, 2020 at 12:30:30PM +0200, Richard Biener wrote: >>> >> I think this is the wrong way to approach this. You're doing too many >>> >> things at once. Try to fix the powerpc regression with the

[Bug c/95257] valgrind gives me error when fwprintf in stderr

2020-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95257 Andrew Pinski changed: What|Removed |Added Resolution|--- |MOVED Status|UNCONFIRMED

[Bug target/95266] New: [11 regression] gcc.dg/vect/bb-slp-pr69907.c fails on power 7

2020-05-21 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95266 Bug ID: 95266 Summary: [11 regression] gcc.dg/vect/bb-slp-pr69907.c fails on power 7 Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

[Bug tree-optimization/95199] Remove extra variable created for memory reference in loop vectorization.

2020-05-21 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95199 --- Comment #5 from bin cheng --- (In reply to Richard Biener from comment #1) > But IVOPTs is supposed to know how to eliminate equal IVs. Maybe it's > confused > about the IFN uses? It's an known issue that IVOPTs has difficulty in

[Bug sanitizer/95137] Sanitizers seem to be missing support for coroutines

2020-05-21 Thread rafael at espindo dot la
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95137 --- Comment #20 from Rafael Avila de Espindola --- The attached testcase also fails with just -fsanitize=undefined. I have tested with gcc version gcc (GCC) 10.1.1 20200507 (Red Hat 10.1.1-1)

[Bug sanitizer/95137] Sanitizers seem to be missing support for coroutines

2020-05-21 Thread rafael at espindo dot la
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95137 Rafael Avila de Espindola changed: What|Removed |Added Attachment #48547|0 |1 is obsolete|

Re: drop -aux{dir,base}, revamp -dump{dir,base}

2020-05-21 Thread Alexandre Oliva
On May 19, 2020, Richard Biener wrote: > On Tue, 19 May 2020, Alexandre Oliva wrote: >> I've refreshed the patch, approved back on Jan 22 for gcc-11, in >> refs/users/aoliva/heads/aux-dump-revamp, and committed 3 other related >> patches on top of it, that I hope to get approved for folding and

RFC: Provide diagnostic hints for missing inttypes.h string constants.

2020-05-21 Thread Mark Wielaard
This is on top of the stdbool.h and stdint.h patches. This adds a flag to c_parser so we know when we were trying to constract a string literal. If there is a parse error and we were constructing a string literal, and the next token is an unknown identifier name, and we know there is a standard

[Bug target/95265] aarch64: suboptimal code generation for common neon intrinsic sequence involving shrn and mull

2020-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95265 Andrew Pinski changed: What|Removed |Added Keywords||missed-optimization

gcc-8-20200521 is now available

2020-05-21 Thread GCC Administrator via Gcc
Snapshot gcc-8-20200521 is now available on https://gcc.gnu.org/pub/gcc/snapshots/8-20200521/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 8 git branch with the following options: git://gcc.gnu.org/git/gcc.git branch

[Bug target/95265] New: aarch64: suboptimal code generation for common neon intrinsic sequence involving shrn and mull

2020-05-21 Thread generictoadhuman at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95265 Bug ID: 95265 Summary: aarch64: suboptimal code generation for common neon intrinsic sequence involving shrn and mull Product: gcc Version: 10.1.0 Status: UNCONFIRMED

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Ian Lance Taylor via Gcc
On Tue, May 19, 2020 at 2:26 AM Martin Liška wrote: > > We've just installed server git hooks that verify git messages > for a correct ChangeLog format. For a limited time period, please > still apply ChangeLog changes to the corresponding ChangeLog files. > We'll use it for comparison of

Re: [PATCH] gcc/Makefile.in: move SELFTEST_DEPS before including language makefile fragments

2020-05-21 Thread David Malcolm via Gcc-patches
On Thu, 2020-05-21 at 17:35 +0200, Romain Naour wrote: > As reported by several Buildroot users [1][2][3], the gcc build > may fail while running selftests makefile target. > > The problem only occurs when ccache is used with gcc 9 and 10, > probably due to a race condition. > > While debuging

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Ian Lance Taylor via Gcc-patches
On Tue, May 19, 2020 at 2:26 AM Martin Liška wrote: > > We've just installed server git hooks that verify git messages > for a correct ChangeLog format. For a limited time period, please > still apply ChangeLog changes to the corresponding ChangeLog files. > We'll use it for comparison of

[Bug c++/95264] New: Infinite Loop When Compiling Templated C++ code at -O1 and above

2020-05-21 Thread freddie at witherden dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95264 Bug ID: 95264 Summary: Infinite Loop When Compiling Templated C++ code at -O1 and above Product: gcc Version: 10.1.0 Status: UNCONFIRMED Severity: normal

Re: New mklog script

2020-05-21 Thread Jason Merrill via Gcc
On Fri, May 15, 2020 at 11:39 AM Martin Liška wrote: > > On 5/15/20 3:22 PM, Marek Polacek wrote: > > On Fri, May 15, 2020 at 03:12:27PM +0200, Martin Liška wrote: > >> On 5/15/20 2:42 PM, Marek Polacek wrote: > >>> I actually use mklog -i all the time. But I can work around it if it > >>>

Re: New mklog script

2020-05-21 Thread Jason Merrill via Gcc-patches
On Fri, May 15, 2020 at 11:39 AM Martin Liška wrote: > > On 5/15/20 3:22 PM, Marek Polacek wrote: > > On Fri, May 15, 2020 at 03:12:27PM +0200, Martin Liška wrote: > >> On 5/15/20 2:42 PM, Marek Polacek wrote: > >>> I actually use mklog -i all the time. But I can work around it if it > >>>

[Bug c++/95263] [11 Regression] ICE in lookup_template_class_1 since r11-504-g74744bb1f2847b5b

2020-05-21 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95263 Nathan Sidwell changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c++/95263] [11 Regression] ICE in lookup_template_class_1 since r11-504-g74744bb1f2847b5b

2020-05-21 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95263 Martin Liška changed: What|Removed |Added Summary|ice in |[11 Regression] ICE in

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 11:01 PM, Jason Merrill wrote: Why?  What is the use of requiring ChangeLog entries at all for these changes? I must confirm a common test-suite ChangeLog entry is something like: $ grep ':' gcc/testsuite/ChangeLog | sed 's/.*://' | sort | uniq -c | sort -n | tac | head -n 15

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 11:01 PM, Jason Merrill wrote: Why?  What is the use of requiring ChangeLog entries at all for these changes? I must confirm a common test-suite ChangeLog entry is something like: $ grep ':' gcc/testsuite/ChangeLog | sed 's/.*://' | sort | uniq -c | sort -n | tac | head -n 15

[Bug c++/95263] New: ice in lookup_template_class_1

2020-05-21 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95263 Bug ID: 95263 Summary: ice in lookup_template_class_1 Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc-patches
On Thu, May 21, 2020 at 4:27 PM Martin Liška wrote: > On 5/21/20 9:51 PM, Jason Merrill wrote: > > Modified. Adjustments to expected errors in testcases don't seem to me > worth documenting in a ChangeLog. > > I see. As Jakub mentioned, I would keep the hook stricter for now. > Why? What is

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc
On Thu, May 21, 2020 at 4:27 PM Martin Liška wrote: > On 5/21/20 9:51 PM, Jason Merrill wrote: > > Modified. Adjustments to expected errors in testcases don't seem to me > worth documenting in a ChangeLog. > > I see. As Jakub mentioned, I would keep the hook stricter for now. > Why? What is

[Bug fortran/95106] Bogus warning from module with long name and an equivalence

2020-05-21 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95106 --- Comment #2 from anlauf at gcc dot gnu.org --- Something like diff --git a/gcc/fortran/trans-common.c b/gcc/fortran/trans-common.c index bf163bc4f52..06313873002 100644 --- a/gcc/fortran/trans-common.c +++ b/gcc/fortran/trans-common.c @@

Writing automated tests for the GCC driver

2020-05-21 Thread Giuliano Belinassi via Gcc
Hi, all. GCC have a extensive testsuite, that is no news at all. However they are focused on the compiler (cc1*) or in libraries, and I can't find tests related to the GCC driver. Are there tests to the GCC driver? If yes, is there any docs about how to write them? Thank you, Giuliano.

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 9:51 PM, Jason Merrill wrote: Modified.  Adjustments to expected errors in testcases don't seem to me worth documenting in a ChangeLog. I see. As Jakub mentioned, I would keep the hook stricter for now. Martin

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 9:51 PM, Jason Merrill wrote: Modified.  Adjustments to expected errors in testcases don't seem to me worth documenting in a ChangeLog. I see. As Jakub mentioned, I would keep the hook stricter for now. Martin

Re: [stage1][PATCH] Lower VEC_COND_EXPR into internal functions.

2020-05-21 Thread Segher Boessenkool
Hi! On Thu, May 21, 2020 at 03:29:49PM +0200, Martin Liška wrote: > Adding Segher to CC, he can help us. Oh dear. Are you sure? > On 5/21/20 2:51 PM, Martin Liška wrote: > >Back to this I noticed that ppc64le target build is broken due to: > >insn-emit.o -MMD -MP -MF ./.deps/insn-emit.TPo

[Bug fortran/95106] Bogus warning from module with long name and an equivalence

2020-05-21 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95106 anlauf at gcc dot gnu.org changed: What|Removed |Added CC||anlauf at gcc dot gnu.org

Re: Welcome GCC GSoC 2020 participants

2020-05-21 Thread Martin Jambor
Hello Tony, On Wed, May 20 2020, y2s1982 . wrote: > Hello Martin, > On Mon, May 18, 2020 at 11:32 AM Martin Jambor wrote: > >> Hello Tony, >> >> sorry for not getting back to you last week. Time seems to fly even >> faster now that I'm forced to work from home :-/ Furthermore, both me >> and

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc-patches
On Thu, May 21, 2020 at 2:58 PM Martin Liška wrote: > On 5/21/20 8:52 PM, Jason Merrill wrote: > > Was there a decision somewhere to require ChangeLog entries for all > testcase changes now, as the hook is enforcing? They were optional before. > > Right now we ignore newly added test-case,

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc
On Thu, May 21, 2020 at 2:58 PM Martin Liška wrote: > On 5/21/20 8:52 PM, Jason Merrill wrote: > > Was there a decision somewhere to require ChangeLog entries for all > testcase changes now, as the hook is enforcing? They were optional before. > > Right now we ignore newly added test-case,

[pushed] coroutines: Partial reversion of r11-437-g5ef067eb14d4.

2020-05-21 Thread Iain Sandoe
Hi I got a bit over-zealous in the attempt to make better use of the higher- level APIs in the c++ FE, this reverts that part of the change, Tested on x86_64-darwin, applied to master, thanks Iain co_returns are statements, not expressions; they do not need to be wrapped in an

[Bug c++/79627] Ice with type of VLA in lambda

2020-05-21 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79627 Eric Gallager changed: What|Removed |Added CC||egallager at gcc dot gnu.org

[Bug libstdc++/95209] std::filesystem::path::lexically_normal mangles "//foo"

2020-05-21 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95209 --- Comment #5 from Jonathan Wakely --- (In reply to M Welinder from comment #4) > Sorry about the missing "-v". It is indeed a x86_64-suse-linux system. > (It's not internet facing or I'd go get the full output.) > > "Implementation

[Bug libfortran/95195] gfortran poorly handles a program error of writing a namelist to an unformatted file.

2020-05-21 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95195 anlauf at gcc dot gnu.org changed: What|Removed |Added Component|fortran |libfortran

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 8:52 PM, Jason Merrill wrote: Was there a decision somewhere to require ChangeLog entries for all testcase changes now, as the hook is enforcing?  They were optional before. Right now we ignore newly added test-case, these don't have to be mentioned. Can you please attach the patch

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 8:52 PM, Jason Merrill wrote: Was there a decision somewhere to require ChangeLog entries for all testcase changes now, as the hook is enforcing?  They were optional before. Right now we ignore newly added test-case, these don't have to be mentioned. Can you please attach the patch

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jakub Jelinek via Gcc
On Thu, May 21, 2020 at 02:52:37PM -0400, Jason Merrill wrote: > Was there a decision somewhere to require ChangeLog entries for all > testcase changes now, as the hook is enforcing? They were optional before. > > remote: *** ChangeLog format failed: > remote: ERR: changed file not mentioned in

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jakub Jelinek via Gcc-patches
On Thu, May 21, 2020 at 02:52:37PM -0400, Jason Merrill wrote: > Was there a decision somewhere to require ChangeLog entries for all > testcase changes now, as the hook is enforcing? They were optional before. > > remote: *** ChangeLog format failed: > remote: ERR: changed file not mentioned in

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc
Was there a decision somewhere to require ChangeLog entries for all testcase changes now, as the hook is enforcing? They were optional before. remote: *** ChangeLog format failed: remote: ERR: changed file not mentioned in a ChangeLog:"gcc/testsuite/g++.dg/parse/error33.C" On Thu, May 21, 2020

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Jason Merrill via Gcc-patches
Was there a decision somewhere to require ChangeLog entries for all testcase changes now, as the hook is enforcing? They were optional before. remote: *** ChangeLog format failed: remote: ERR: changed file not mentioned in a ChangeLog:"gcc/testsuite/g++.dg/parse/error33.C" On Thu, May 21, 2020

[pushed] c++: Check constant array bounds later.

2020-05-21 Thread Jason Merrill via Gcc-patches
We give a better diagnostic for non-constant array bounds in compute_array_index_type_loc, we don't need to diagnose it in the parser. But to avoid a regression on parse/varmod1.C we need to actually check non-dependent expressions in a template. Tested x86_64-pc-linux-gnu, applying to trunk.

[pushed] c++: Improve error recovery for =.

2020-05-21 Thread Jason Merrill via Gcc-patches
In a template we were happily embedding error_mark_node in a MODOP_EXPR, leading to confusion later. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/cp/ChangeLog 2020-05-21 Jason Merrill * typeck.c (build_x_modify_expr): Handle error_mark_node arguments. --- gcc/cp/typeck.c

[pushed] c++: Constant expression parsing and parameters.

2020-05-21 Thread Jason Merrill via Gcc-patches
The difference between a "potential" constant-expression and a regular constant-expression is the treatment of parameters; in a constexpr function, a parameter is potentially constant when evaluating a call to that function, but it is not constant during parsing of the function.

[pushed] c++: Improve error-recovery for parms.

2020-05-21 Thread Jason Merrill via Gcc-patches
If a parameter is erroneous, we currently drop it, leading to "too many arguments" errors later. Treating the function as (...) avoids those errors. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/cp/ChangeLog 2020-05-21 Jason Merrill * decl.c (grokparms): Return NULL_TREE if any

[Bug target/95169] [10/11 Regression] i386 comparison between nan and 0.0 triggers Invalid operation exception

2020-05-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95169 Uroš Bizjak changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

Re: [PATCH] Fix handling of OPT_mgeneral_regs_only in attribute.

2020-05-21 Thread Martin Liška
Ah, you are right. Please forget about this patch mgeneral-regs-only Target Report RejectNegative Mask(GENERAL_REGS_ONLY) Save also contains the RejectNegative keyword. Martin

[Bug target/95256] [11 Regression] ICE in convert_move, at expr.c:278 since r11-263-g7c355156aa20eaec

2020-05-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95256 Uroš Bizjak changed: What|Removed |Added Depends on||95125 CC|

[Bug c++/95262] New: Taking address of function pointer do full concept overload resolution

2020-05-21 Thread barry.revzin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95262 Bug ID: 95262 Summary: Taking address of function pointer do full concept overload resolution Product: gcc Version: 10.0 Status: UNCONFIRMED Severity: normal

[Bug target/95256] [11 Regression] ICE in convert_move, at expr.c:278 since r11-263-g7c355156aa20eaec

2020-05-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95256 Uroš Bizjak changed: What|Removed |Added Target Milestone|--- |11.0 --- Comment #2 from Uroš Bizjak ---

[Bug target/95229] [11 Regression] in mark_jump_label_1

2020-05-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95229 Uroš Bizjak changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |ubizjak at gmail dot com

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2020-05-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 95229, which changed state. Bug 95229 Summary: [11 Regression] in mark_jump_label_1 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95229 What|Removed |Added

[Bug target/95261] New: [11 regression] gcc.target/powerpc/pr80695-p8.c and gcc.target/powerpc/pr80695-p9.c fail starting with r11-478

2020-05-21 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95261 Bug ID: 95261 Summary: [11 regression] gcc.target/powerpc/pr80695-p8.c and gcc.target/powerpc/pr80695-p9.c fail starting with r11-478 Product: gcc Version: 11.0

Re: [PATCH] gcc/Makefile.in: move SELFTEST_DEPS before including language makefile fragments

2020-05-21 Thread Jeff Law via Gcc-patches
On Thu, 2020-05-21 at 19:31 +0200, Richard Biener via Gcc-patches wrote: > On May 21, 2020 5:35:19 PM GMT+02:00, Romain Naour via Gcc-patches < > gcc-patches@gcc.gnu.org> wrote: > > As reported by several Buildroot users [1][2][3], the gcc build > > may fail while running selftests makefile

V2 [PATCH] x86: Move cpuinfo.h from libgcc to common/config/i386

2020-05-21 Thread H.J. Lu via Gcc-patches
On Wed, May 20, 2020 at 4:21 AM H.J. Lu wrote: > > On Tue, May 19, 2020 at 11:10 PM Uros Bizjak wrote: > > > > On Tue, May 19, 2020 at 11:40 PM H.J. Lu wrote: > > > > > > > > > > > > I will take a look to see if we share the same CPU > > > > > > > > > > detection code between > > > > > > > > >

[Bug target/95169] [10/11 Regression] i386 comparison between nan and 0.0 triggers Invalid operation exception

2020-05-21 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95169 --- Comment #8 from CVS Commits --- The releases/gcc-10 branch has been updated by Uros Bizjak : https://gcc.gnu.org/g:ddfb80adbd91a0e45cc6e04a8b0dbe3ca15ba45f commit r10-8167-gddfb80adbd91a0e45cc6e04a8b0dbe3ca15ba45f Author: Uros Bizjak

RE: [PATCH] Fix handling of OPT_mgeneral_regs_only in attribute.

2020-05-21 Thread Sudakshina Das
Hi Martin > -Original Message- > From: Martin Liška > Sent: 21 May 2020 16:01 > To: gcc-patches@gcc.gnu.org > Cc: Sudakshina Das > Subject: [PATCH] Fix handling of OPT_mgeneral_regs_only in attribute. > > Hi. > > Similarly to: > > case OPT_mstrict_align: >if (val) >

Re: [PATCH] gcc/Makefile.in: move SELFTEST_DEPS before including language makefile fragments

2020-05-21 Thread Richard Biener via Gcc-patches
On May 21, 2020 5:35:19 PM GMT+02:00, Romain Naour via Gcc-patches wrote: >As reported by several Buildroot users [1][2][3], the gcc build >may fail while running selftests makefile target. > >The problem only occurs when ccache is used with gcc 9 and 10, >probably due to a race condition. Just

[Bug sanitizer/95244] GCC 10 no longer builds on RHEL5 [trivial patch]

2020-05-21 Thread lopresti at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95244 --- Comment #2 from Patrick J. LoPresti --- Done (https://github.com/google/sanitizers/issues/1253). Thanks.

[Bug libstdc++/95209] std::filesystem::path::lexically_normal mangles "//foo"

2020-05-21 Thread terra at gnome dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95209 --- Comment #4 from M Welinder --- Sorry about the missing "-v". It is indeed a x86_64-suse-linux system. (It's not internet facing or I'd go get the full output.) "Implementation defined", yes, but the implementation is the os, not the

Re: [PATCH] Fix handling of OPT_mgeneral_regs_only in attribute.

2020-05-21 Thread H.J. Lu via Gcc-patches
On Thu, May 21, 2020 at 9:22 AM Martin Liška wrote: > > Hi. > > Similarly to: > > case OPT_mstrict_align: >if (val) > opts->x_target_flags |= MASK_STRICT_ALIGN; >else > opts->x_target_flags &= ~MASK_STRICT_ALIGN; >return true; > > the

Re: [PATCH] Let numeric_limits::is_iec559 reflect -ffast-math

2020-05-21 Thread Jonathan Wakely via Gcc-patches
On 21/05/20 17:46 +0200, Marc Glisse wrote: On Thu, 21 May 2020, Jonathan Wakely wrote: On 27/04/20 17:09 +0200, Matthias Kretz wrote: From: Matthias Kretz PR libstdc++/84949 * include/std/limits: Let is_iec559 reflect whether __GCC_IEC_559 says float and double support

Re: [PATCH] Let numeric_limits::is_iec559 reflect -ffast-math

2020-05-21 Thread Marc Glisse
On Thu, 21 May 2020, Jonathan Wakely wrote: On 27/04/20 17:09 +0200, Matthias Kretz wrote: From: Matthias Kretz PR libstdc++/84949 * include/std/limits: Let is_iec559 reflect whether __GCC_IEC_559 says float and double support IEEE 754-2008. *

Re: 1-800-GIT-HELP Reverting a patch

2020-05-21 Thread Thomas Koenig via Gcc
Hi Andreas, git is not doing a plain patch, it is doing a merge. It is not unusual for a merge to have changes that are already present on both sides. ... which just goes to show that it is very easy to make a fool of yourself with git if you have no mental model of what it does. That's not

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Rainer Orth
Hi Martin, >> two comments: >> >> * Can you please avoid the use grey highlighting in that section? Black >>script on a grey background is already hard to read for someone with >>reasonable vision. I suspect it will be much harder for >>vision-impaired people. > > You are right, I

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Rainer Orth
Hi Martin, >> two comments: >> >> * Can you please avoid the use grey highlighting in that section? Black >>script on a grey background is already hard to read for someone with >>reasonable vision. I suspect it will be much harder for >>vision-impaired people. > > You are right, I

[PATCH] gcc/Makefile.in: move SELFTEST_DEPS before including language makefile fragments

2020-05-21 Thread Romain Naour via Gcc-patches
As reported by several Buildroot users [1][2][3], the gcc build may fail while running selftests makefile target. The problem only occurs when ccache is used with gcc 9 and 10, probably due to a race condition. While debuging with "make -p" we can notice that s-selftest-c target contain only

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 5:14 PM, Rainer Orth wrote: Hi Martin, We've just installed server git hooks that verify git messages for a correct ChangeLog format. For a limited time period, please still apply ChangeLog changes to the corresponding ChangeLog files. We'll use it for comparison of auto-generated

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Martin Liška
On 5/21/20 5:14 PM, Rainer Orth wrote: Hi Martin, We've just installed server git hooks that verify git messages for a correct ChangeLog format. For a limited time period, please still apply ChangeLog changes to the corresponding ChangeLog files. We'll use it for comparison of auto-generated

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Rainer Orth
Hi Martin, > We've just installed server git hooks that verify git messages > for a correct ChangeLog format. For a limited time period, please > still apply ChangeLog changes to the corresponding ChangeLog files. > We'll use it for comparison of auto-generated CangeLog entries. > > The format is

Re: ChangeLog files - server and client scripts

2020-05-21 Thread Rainer Orth
Hi Martin, > We've just installed server git hooks that verify git messages > for a correct ChangeLog format. For a limited time period, please > still apply ChangeLog changes to the corresponding ChangeLog files. > We'll use it for comparison of auto-generated CangeLog entries. > > The format is

[COMMITTED] Include memmodel.h in adjust-alignment.c

2020-05-21 Thread Rainer Orth
Hi Kito, > Tested and committed with fixes, thanks your review :) this patch broke SPARC bootstrap: In file included from ./tm_p.h:4, from /vol/gcc/src/hg/master/local/gcc/adjust-alignment.c:28: /vol/gcc/src/hg/master/local/gcc/config/sparc/sparc-protos.h:45:47: error: use of

[PATCH] Fix handling of OPT_mgeneral_regs_only in attribute.

2020-05-21 Thread Martin Liška
Hi. Similarly to: case OPT_mstrict_align: if (val) opts->x_target_flags |= MASK_STRICT_ALIGN; else opts->x_target_flags &= ~MASK_STRICT_ALIGN; return true; the MASK_GENERAL_REGS_ONLY mask should be handled the same way. @Sudakshina: The

[Bug target/95260] Confusing comments in cpuid.h

2020-05-21 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95260 --- Comment #4 from H.J. Lu --- (In reply to Martin Liška from comment #1) > Just a question: why do you create PRs for all these issues? > Is it because you want to backport fixes to active branches? That would be nice.

[Bug target/95259] Duplicated codes in libgcc, driver-i386.c and i386-builtins.c

2020-05-21 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95259 H.J. Lu changed: What|Removed |Added Attachment #48576|0 |1 is obsolete|

[Bug target/95260] Confusing comments in cpuid.h

2020-05-21 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95260 H.J. Lu changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

Re: [PATCH] Implement no_stack_protect attribute.

2020-05-21 Thread Martin Sebor via Gcc-patches
On 5/21/20 5:28 AM, Martin Liška wrote: On 5/18/20 10:37 PM, Martin Sebor wrote: I know there are some somewhat complex cases the attribute exclusion mechanism isn't general enough to handle but this seems simple enough that it should work.  Unless I'm missing something that makes it not

[Bug target/95260] Confusing comments in cpuid.h

2020-05-21 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95260 --- Comment #2 from CVS Commits --- The master branch has been updated by H.J. Lu : https://gcc.gnu.org/g:a74630f3207d0bec63710c8c803685a0a4956986 commit r11-552-ga74630f3207d0bec63710c8c803685a0a4956986 Author: H.J. Lu Date: Thu May 21

[Bug target/95260] Confusing comments in cpuid.h

2020-05-21 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95260 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #1

Re: [PATCH] Let numeric_limits::is_iec559 reflect -ffast-math

2020-05-21 Thread Jonathan Wakely via Gcc-patches
On 27/04/20 17:09 +0200, Matthias Kretz wrote: From: Matthias Kretz PR libstdc++/84949 * include/std/limits: Let is_iec559 reflect whether __GCC_IEC_559 says float and double support IEEE 754-2008. * testsuite/18_support/numeric_limits/is_iec559.cc: Test IEC559

[PATCH] config/i386/cpuid.h: Use hexadecimal in comments

2020-05-21 Thread H.J. Lu via Gcc-patches
Since Intel SDM uses hexadecimal, use hexadecimal in comments. PR target/95260 * config/i386/cpuid.h: Use hexadecimal in comments. --- gcc/ChangeLog | 5 + gcc/config/i386/cpuid.h | 6 +++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Richard Sandiford
Martin Liška writes: > On 5/21/20 12:11 PM, Richard Sandiford wrote: >> Yeah, agree it'd be worth having tests for both directions. The patch >> itself looks good though -- thanks for doing this. > > Thanks. There's a version with 2 new tests that I've just tested. > > I'm going to install the

[Bug target/95260] New: Confusing comments in cpuid.h

2020-05-21 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95260 Bug ID: 95260 Summary: Confusing comments in cpuid.h Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target

  1   2   >