Re: [PATCH] profopt-execute: unset testname_with_flags if create_gcov fails

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/26/20 12:22 PM, Ilya Leoshkevich via Gcc-patches wrote: > Bootstrapped and regtested on x86_64-redhat-linux and > s390x-redhat-linux. Ok for master? > > > > When diffing test results, there sometimes occur spurious "New tests > that PASS" / "Old tests that passed, that have disappeared"

[Bug c++/97962] ICE in build_over_call, at cp/call.c:8976

2020-11-30 Thread mpolacek at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97962 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org

[Bug c++/80780] Front-end support needed for experimental::source_location

2020-11-30 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80780 --- Comment #9 from Jakub Jelinek --- Created attachment 49653 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49653=edit gcc11-pr80780-wip.patch For the default argument, my current ugly hack is attached. But guess it needs to be done

Re: Possible code to remove DECL_NONSHAREABLE?

2020-11-30 Thread Jeff Law via Gcc
On 11/27/20 5:47 AM, Matthew Malcomson via Gcc wrote: > Hi there, > > I was just looking through the history of how some code came about, > and get the impression that DECL_NONSHAREABLE was meant to be removed. > > It seems like it was added to solve PR49103, with the idea that it > could be

Re: [PATCH] gcc-11/changes: Document new configure flag --enable-s390-excess-float-precision

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/26/20 2:58 AM, Marius Hillenbrand via Gcc-patches wrote: > Hi, > > To document the new behavior around FLT_EVAL_METHOD and configure flag > --enable-s390-excess-float-precision on s390, I propose this update to the > Release Notes. Please commit to git-wwwdocs if you agree. > > Checked

[Bug c++/80780] Front-end support needed for experimental::source_location

2020-11-30 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80780 --- Comment #8 from Jonathan Wakely --- The default argument case is: source_location f(source_location a = source_location::current()) { return a; } int main() { auto loc = f(); // f's first argument corresponds to this line of code

Re: [PATCH] [tree-optimization] Optimize max/min pattern with comparison

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/25/20 3:04 PM, Eugene Rozenfeld via Gcc-patches wrote: > Make the following simplifications: > X <= MAX(X, Y) -> true > X > MAX(X, Y) -> false > X >= MIN(X, Y) -> true > X < MIN(X, Y) -> false > > This fixes PR96708. > > Tested on x86_64-pc-linux-gnu. > > bool

Re: [PATCH] dse: Cope with bigger-than-integer modes [PR98037]

2020-11-30 Thread Richard Biener via Gcc-patches
On November 30, 2020 4:29:41 PM GMT+01:00, Richard Sandiford via Gcc-patches wrote: >dse.c:find_shift_sequence tries to represent a store and load >back as a shift right followed by a truncation. It therefore >needs to find an integer mode in which to do the shift right. >The loop it uses has

Re: [PATCH] ipa-cp: Avoid unwanted multiple propagations (PR 97816)

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/20/20 12:41 PM, Martin Jambor wrote: > Hi, > > this is an updated patch based on our conversation on IRC today. So far > I have had a look at the effects on only tramp3d and although it makes > the heuristics more pessimistic more times than optimistic (number of > clones at -Ofast drops

[Bug c/97172] [11 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams since r11-3303-g6450f07388f9fe57

2020-11-30 Thread msebor at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97172 --- Comment #14 from Martin Sebor --- I submitted a simple patch to do the unsharing as the first step toward fixing this bug here: https://gcc.gnu.org/pipermail/gcc-patches/2020-November/559770.html I'm not sure I understand correctly what you

Re: How to traverse all the local variables that declared in the current routine?

2020-11-30 Thread Martin Sebor via Gcc-patches
On 11/30/20 9:23 AM, Qing Zhao wrote: Hi, Martin, Thanks a lot for your suggestion. On Nov 25, 2020, at 6:08 PM, Martin Sebor > wrote: On 11/24/20 9:54 AM, Qing Zhao via Gcc-patches wrote: On Nov 24, 2020, at 9:55 AM, Richard Biener

Re: [PATCH] driver: Don't imply -dD for -g3 -g0 [PR97989]

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/26/20 1:39 AM, Jakub Jelinek via Gcc-patches wrote: > Hi! > > The driver enables -dD when preprocessing when -g3 is specified, for obvious > reasons > that we need the macros to be preserved somewhere for them to make up the > debug > info. But it enables it even if -g3 is later

[Bug c++/97995] [8/9/10/11 Regression] ICE tree check: expected tree that contains 'typed' structure, have 'deferred_noexcept' in unify, at cp/pt.c:23473 since r7-4383-g51dc660315ef83dc

2020-11-30 Thread mpolacek at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97995 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

[Bug rtl-optimization/98037] ICE in dse.c:find_shift_sequence for large non-integer modes

2020-11-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98037 --- Comment #1 from CVS Commits --- The master branch has been updated by Richard Sandiford : https://gcc.gnu.org/g:f835e9f6562dda9c8a1384be2c9d4e45c112ed8e commit r11-5580-gf835e9f6562dda9c8a1384be2c9d4e45c112ed8e Author: Richard Sandiford

[Bug c++/80780] Front-end support needed for experimental::source_location

2020-11-30 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80780 --- Comment #7 from Jonathan Wakely --- Oh great, thanks :-) I might push my implementation of std::source_location, with some tests commented out for now.

Re: [PATCH v2] C-family : Add attribute 'unavailable'.

2020-11-30 Thread Martin Sebor via Gcc-patches
On 11/29/20 6:56 PM, Iain Sandoe wrote: Hi Martin, Martin Sebor via Gcc-patches wrote: On 11/10/20 12:38 PM, Iain Sandoe wrote: —— commit message. If an interface is marked 'deprecated' then, presumably, at some point it will be withdrawn and no longer available.  The 'unavailable'

Re: [PATCH] dse: Cope with bigger-than-integer modes [PR98037]

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/30/20 8:29 AM, Richard Sandiford via Gcc-patches wrote: > dse.c:find_shift_sequence tries to represent a store and load > back as a shift right followed by a truncation. It therefore > needs to find an integer mode in which to do the shift right. > The loop it uses has the form: > >

[Bug middle-end/19987] [meta-bug] fold missing optimizations in general

2020-11-30 Thread law at redhat dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19987 Bug 19987 depends on bug 96679, which changed state. Bug 96679 Summary: Failure to optimize or+and+or pattern to and+or https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96679 What|Removed |Added

[Bug tree-optimization/96679] Failure to optimize or+and+or pattern to and+or

2020-11-30 Thread law at redhat dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96679 Jeffrey A. Law changed: What|Removed |Added CC||law at redhat dot com

Re: [PATCH] [tree-optimization] Optimize or+and+or pattern to and+or

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/22/20 6:35 PM, Eugene Rozenfeld via Gcc-patches wrote: > Simplify > ((b | c) & a) | b > to > (a & c) | b. > > This fixes PR96679. > > Tested on x86_64-pc-linux-gnu. > > int f(int a, int b, int c) > { > return ((b | c) & a) | b; > } > > Code without the patch: > or edx,esi > and

[Bug tree-optimization/96679] Failure to optimize or+and+or pattern to and+or

2020-11-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96679 --- Comment #1 from CVS Commits --- The master branch has been updated by Jeff Law : https://gcc.gnu.org/g:28a7fdd81e857057f18f87a3c9dd180ad99b77f6 commit r11-5579-g28a7fdd81e857057f18f87a3c9dd180ad99b77f6 Author: Eugene Rozenfeld Date: Mon

Re: [PATCH v3 1/2] generate EH info for volatile asm statements (PR93981)

2020-11-30 Thread J.W. Jagersma via Gcc-patches
On 2020-11-23 09:20, Richard Biener wrote: > On Sun, Nov 22, 2020 at 5:38 PM J.W. Jagersma wrote: >> >> On 2020-11-21 12:27, J.W. Jagersma wrote: >>> ... >>> Another idea I had is to introduce a new operand modifier, eg. '-', which >>> would signify that the output *must* be considered clobbered

Re: [RFC] Increase libstdc++ line length to 100(?) columns

2020-11-30 Thread Michael Matz
Hello, On Mon, 30 Nov 2020, Allan Sandfeld Jensen wrote: > > > On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote: > > > > * Allan Sandfeld Jensen: > > > > > If you _do_ change it. I would suggest changing it to 120, which is > > > > > next > > > > > common step for a lot of C++

Re: [PATCH][AVX512]Lower AVX512 vector compare to AVX version when dest is vector

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/16/20 8:10 PM, Hongtao Liu wrote: > On Tue, Nov 17, 2020 at 8:05 AM Jeff Law wrote: >> >> On 9/2/20 3:34 AM, Hongtao Liu via Gcc-patches wrote: >>> Hi: >>> Add define_peephole2 to eliminate potential redundant conversion >>> from mask to vector. >>> Bootstrap is ok, regression test

[Bug c/98070] New: errno is not re-evaluated after clearing errno and calling realloc(ptr, SIZE_MAX)

2020-11-30 Thread stli at linux dot ibm.com via Gcc-bugs
--disable-bootstrap --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-linker-hash-style=gnu --enable-plugin --enable-initfini-array --disable-libgcj --disable-multilib Thread model: posix Supported LTO compression algorithms: zlib zstd gcc vers

Re: [PATCH] ipa: dump symtab to emergency dump file

2020-11-30 Thread Martin Liška
On 11/30/20 4:50 PM, Jan Hubicka wrote: On 11/30/20 7:04 AM, Martin Liška wrote: Hi. It's handy to have symbol table when we dump emergency dump. It's likely next stage1 material. Thoughts? Martin gcc/ChangeLog:     * passes.c (emergency_dump_function): Dump symtab when     we

[Bug tree-optimization/98069] [8/9/10/11 Regression] Miscompilation with -O3 since r8-2380-g2d7744d4ef93bfff

2020-11-30 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98069 Martin Liška changed: What|Removed |Added Known to work||7.4.0 Known to fail|

[Bug tree-optimization/98069] [8/9/10/11 Regression] Miscompilation with -O3 since r8-2380-g2d7744d4ef93bfff

2020-11-30 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98069 Martin Liška changed: What|Removed |Added Summary|Miscompilation with -O3 |[8/9/10/11 Regression]

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

2020-11-30 Thread abebeos at lazaridis dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729 --- Comment #26 from abebeos at lazaridis dot com --- (In reply to Richard Biener from comment #24) > Amending / adjusting > https://gcc.gnu.org/wiki/Building_Cross_Toolchains_with_gcc > (the only place that somewhat "documents" how to setup AVR

Re: [RFC] Increase libstdc++ line length to 100(?) columns

2020-11-30 Thread Allan Sandfeld Jensen
On Montag, 30. November 2020 16:47:08 CET Michael Matz wrote: > Hello, > > On Sun, 29 Nov 2020, Allan Sandfeld Jensen wrote: > > On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote: > > > * Allan Sandfeld Jensen: > > > > If you _do_ change it. I would suggest changing it to 120, which

[Bug c++/80780] Front-end support needed for experimental::source_location

2020-11-30 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80780 --- Comment #6 from Jakub Jelinek --- I'm aware of https://github.com/cplusplus/draft/pull/3749/commits/ade9b1552eed5b1b0b3fc2808e6575ee6b526301 and am working on that today incidentally.

Re: [PATCH] Remove redundant builtins for avx512f scalar instructions.

2020-11-30 Thread Jakub Jelinek via Gcc-patches
On Mon, Nov 30, 2020 at 09:23:15AM -0700, Jeff Law wrote: > > > On 11/12/20 11:21 PM, Hongyu Wang wrote: > > Hi > > > > Thanks for reminding me about this patch. I didn't remove any existing > > intrinsics, just remove redundant builtin functions that end-users > > would not likely to use. > > >

[Bug c++/80780] Front-end support needed for experimental::source_location

2020-11-30 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80780 Jonathan Wakely changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment

Re: How to traverse all the local variables that declared in the current routine?

2020-11-30 Thread Qing Zhao via Gcc-patches
Hi, Martin, Thanks a lot for your suggestion. > On Nov 25, 2020, at 6:08 PM, Martin Sebor wrote: > > On 11/24/20 9:54 AM, Qing Zhao via Gcc-patches wrote: >>> On Nov 24, 2020, at 9:55 AM, Richard Biener >>> wrote: >>> >>> On Tue, Nov 24, 2020 at 4:47 PM Qing Zhao wrote:

Re: [PATCH] Remove redundant builtins for avx512f scalar instructions.

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/12/20 11:21 PM, Hongyu Wang wrote: > Hi > > Thanks for reminding me about this patch. I didn't remove any existing > intrinsics, just remove redundant builtin functions that end-users > would not likely to use. > > Also I'm OK to keep current implementation, in case there might be >

[Bug target/98060] Failure to optimize cmp+setnb+add to cmp+sbb

2020-11-30 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98060 Uroš Bizjak changed: What|Removed |Added Severity|normal |enhancement Last reconfirmed|

[PATCH] changelog: add hint for a file mismatch

2020-11-30 Thread Martin Liška
Pushed to master. It's supposed to provide a hint, e.g. ERR: unchanged file mentioned in a ChangeLog (did you mean "gcc/testsuite/gfortran.dg/goacc-gomp/free-1.f90"?):"gcc/testsuite/gfortran.dg/goacc-gomp/free-1.f" Martin contrib/ChangeLog: * gcc-changelog/git_commit.py: Suggest

Re: [24/32] module mapper

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/12/20 9:24 AM, Nathan Sidwell wrote: > On 11/3/20 4:17 PM, Nathan Sidwell wrote: >> this is the module mapper client and server pieces.  It features a >> default resolver that can read a text file, or generate default >> mappings from module name to cmi name. > > Richard rightly suggested

Re: [PATCH][PR target/97770] x86: Add missing popcount2 expander

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/11/20 6:54 PM, Hongyu Wang via Gcc-patches wrote: > Hi, > > According to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97770, x86 > backend need popcount2 expander so __builtin_popcount could be > auto vectorized with AVX512BITALG/AVX512VPOPCNTDQ targets. > > For DImode the middle-end

[committed] Fix non-unique testnames

2020-11-30 Thread Jeff Law via Gcc-patches
This patch fixes a handful of tests with non-unique names which confuse the living hell out of compare_tests, particularly if one of two tests [x]fail while the other is [x]pass which compare_tests will flag as a regression each and every run. No doubt there's a lot more of these lying around and

Re: [PATCH 25/31] VAX: Fix predicates for widening multiply and multiply-add insns

2020-11-30 Thread Maciej W. Rozycki
On Fri, 20 Nov 2020, Jeff Law wrote: > ps.  Yes, I skipped the insv/extv changes.  They're usually a rats nest > of special cases.  We'll come back to them. I've thought of actually reducing the number of patterns to the minimum possible by folding the existing ones together, and then getting

[Bug d/98067] [11 Regression] d: ICE in in force_decl_die, at dwarf2out.c:26197 with -gdwarf-2 -gstrict-dwarf

2020-11-30 Thread ibuclaw at gdcproject dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98067 --- Comment #8 from Iain Buclaw --- As a last resort I could just not emit D manifest constants as CONST_DECLs. They are a nice-to-have from the debugger, but functionally equivalent to C macros.

Re: [PATCH] ipa: dump symtab to emergency dump file

2020-11-30 Thread Jan Hubicka
> > > On 11/30/20 7:04 AM, Martin Liška wrote: > > Hi. > > > > It's handy to have symbol table when we dump emergency dump. > > It's likely next stage1 material. > > > > Thoughts? > > Martin > > > > gcc/ChangeLog: > > > >     * passes.c (emergency_dump_function): Dump symtab when > >    

Re: [PATCH] ipa: dump symtab to emergency dump file

2020-11-30 Thread Jeff Law via Gcc-patches
On 11/30/20 7:04 AM, Martin Liška wrote: > Hi. > > It's handy to have symbol table when we dump emergency dump. > It's likely next stage1 material. > > Thoughts? > Martin > > gcc/ChangeLog: > >     * passes.c (emergency_dump_function): Dump symtab when >     we are in an IPA pass. Your

Re: [RFC] Increase libstdc++ line length to 100(?) columns

2020-11-30 Thread Michael Matz
Hello, On Sun, 29 Nov 2020, Allan Sandfeld Jensen wrote: > On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote: > > * Allan Sandfeld Jensen: > > > If you _do_ change it. I would suggest changing it to 120, which is next > > > common step for a lot of C++ projects. > > > > 120 can be

[PATCH] dse: Cope with bigger-than-integer modes [PR98037]

2020-11-30 Thread Richard Sandiford via Gcc-patches
dse.c:find_shift_sequence tries to represent a store and load back as a shift right followed by a truncation. It therefore needs to find an integer mode in which to do the shift right. The loop it uses has the form: FOR_EACH_MODE_FROM (new_mode_iter,

[Bug c/97172] [11 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams since r11-3303-g6450f07388f9fe57

2020-11-30 Thread msebor at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97172 Martin Sebor changed: What|Removed |Added CC||hjl.tools at gmail dot com --- Comment

[Bug lto/98068] [11 Regression] FAIL: gcc.dg/atomic/pr65345-4.c -O2 -flto -fno-use-linker-plugin -flto-partition=none (internal compiler error) by r11-3303

2020-11-30 Thread msebor at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98068 Martin Sebor changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

Re: [PATCH] match.pd: Use ranges to optimize some x * y / y to x [PR97997]

2020-11-30 Thread Andrew MacLeod via Gcc-patches
On 11/26/20 3:52 AM, Jakub Jelinek wrote: Hi! For signed integers with undefined overflow we already optimize x * y / y into x, but for signed integers with -fwrapv or unsigned integers we don't. The following patch allows optimizing that into just x if value ranges prove that x * y will never

[PATCH] VAX: Fix the LTO compiler downgrading code to non-PIC model

2020-11-30 Thread Maciej W. Rozycki
Fix a testsuite failure: /tmp/ccL65Mmt.s: Assembler messages: /tmp/ccL65Mmt.s:36: Warning: Symbol n used as immediate operand in PIC mode. FAIL: gcc.dg/lto/pr55660 c_lto_pr55660_0.o-c_lto_pr55660_1.o link, -O0 -flto -flto-partition=none -fuse-linker-plugin where non-PIC code is substituted by

[Bug target/89057] [8/9/10/11 Regression] AArch64 ld3 st4 less optimized

2020-11-30 Thread abhiraj.garakapati at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89057 Abhiraj Garakapati changed: What|Removed |Added CC||abhiraj.garakapati at gmail dot co

[Bug tree-optimization/98069] New: Miscompilation with -O3

2020-11-30 Thread vsevolod.livinskij at frtk dot ru via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98069 Bug ID: 98069 Summary: Miscompilation with -O3 Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: tree-optimization

[committed] libstdc++: Add new C++20 headers to Doxygen settings

2020-11-30 Thread Jonathan Wakely via Gcc-patches
This doesn't actually have any effect unless you also change the predefined value of __cplusplus, as it's currently 201703L. But if somebody does want to do that, the new headers will get processed now. libstdc++-v3/ChangeLog: * doc/doxygen/user.cfg.in (INPUT): Add and . Tested

Re: [committed] libstdc++: Set dg-timeout-factor for some slow tests

2020-11-30 Thread Jonathan Wakely via Gcc-patches
On 26/11/20 16:25 +, Jonathan Wakely wrote: These tests are very, very slow to compile. If the testsuite is run with a low tool_timeout value they are likely to fail. By adding a multiplication factor to those tests, it's still possible to use a low timeout without spurious failures.

[Bug c/97172] [11 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams since r11-3303-g6450f07388f9fe57

2020-11-30 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97172 --- Comment #12 from Richard Biener --- unsharing the tree is correct AFAICS, avoiding the attribute for VLA types would likely also be good (are those handled in any reasonable way?). Note nothing will update those SSA names so they should not

Re: [RFC] Decrease default timeout for libstdc++ tests to 6 minutes

2020-11-30 Thread Jonathan Wakely via Gcc-patches
On 27/11/20 21:17 +0100, Christophe Lyon via Libstdc++ wrote: On Fri, 27 Nov 2020 at 17:13, Jonathan Wakely via Gcc-patches wrote: The default for the GCC testsuite is 300, i.e. 5 minutes, which is the same as the DejaGnu default. Libstdc++ overrides this to 600, i.e. 10 minutes. This seems

[Bug fortran/98011] [OpenACC] 'gcc/fortran/scanner.c:load_line' should consider 'flag_openacc' in addition to 'flag_openmp' (and vice versa?)?

2020-11-30 Thread burnus at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98011 Tobias Burnus changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

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

2020-11-30 Thread abebeos at lazaridis dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729 --- Comment #25 from abebeos at lazaridis dot com --- (In reply to John Paul Adrian Glaubitz from comment #22) [...] > > https://www.gnu.org/prep/maintain/html_node/Copyright-Papers.html FSF has a fascinating way to make trivial things

[Bug fortran/98010] [OpenACC] 'gcc/fortran/options.c:gfc_post_options' should consider 'flag_openacc' in addition to 'flag_openmp'?

2020-11-30 Thread burnus at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98010 Tobias Burnus changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug fortran/98013] [OpenACC] 'gcc/fortran/trans-decl.c:gfc_generate_function_code' should consider 'flag_openacc' in addition to 'flag_openmp'?

2020-11-30 Thread burnus at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98013 Tobias Burnus changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

Re: Fortran: With OpenACC, ignore OpenMP's cond comp sentinels [PR98011]

2020-11-30 Thread Tobias Burnus
... and now committed as r11-5572-g1d6f6ac693a8601bef9fe4ba72eb6fbf7b60b5cd. Thanks again for the suggestions! Tobias On 27.11.20 23:14, Tobias Burnus wrote: On 27.11.20 18:31, Jakub Jelinek via Fortran wrote: Depends on what does the OpenACC standard say. If it has similar wording to OpenMP

[Bug fortran/98011] [OpenACC] 'gcc/fortran/scanner.c:load_line' should consider 'flag_openacc' in addition to 'flag_openmp' (and vice versa?)?

2020-11-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98011 --- Comment #3 from CVS Commits --- The master branch has been updated by Tobias Burnus : https://gcc.gnu.org/g:1d6f6ac693a8601bef9fe4ba72eb6fbf7b60b5cd commit r11-5572-g1d6f6ac693a8601bef9fe4ba72eb6fbf7b60b5cd Author: Tobias Burnus Date:

Updating the backend status for h8300 on the wiki

2020-11-30 Thread John Paul Adrian Glaubitz
Hi! Now that h8300 has been converted to use MODE_CC, it might be a good idea to update the documentation on the wiki [1] which still lists h8300 as being cc0. Adrian > [1] https://gcc.gnu.org/backends.html -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer - glaub...@debian.org `.

Re: [Patch] Fortran: -fno-automatic and -fopenacc / recusion check cleanup

2020-11-30 Thread Tobias Burnus
Now installed as r11-5571-gf4e7ea81d1369d4d6cb6d8e440aefb3407142e05. Tobias On 27.11.20 11:16, Tobias Burnus wrote: Two fixes – simple, see patch + commit text. Longer description: * options: Background: - OpenMP, OpenACC and imply that a function is called concurrently and -frecursive

[Bug jit/97867] [11 Regression] thunk_info::release breaks function calls in libgccjit

2020-11-30 Thread hubicka at ucw dot cz via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97867 --- Comment #8 from Jan Hubicka --- > > FWIW, did you configure with --enable-host-shared ? Forgetting to enable that > is the usual source of weird errors when building libgccjit. I think it does not let you to build it otherwise, but I

[Bug fortran/98010] [OpenACC] 'gcc/fortran/options.c:gfc_post_options' should consider 'flag_openacc' in addition to 'flag_openmp'?

2020-11-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98010 --- Comment #2 from CVS Commits --- The master branch has been updated by Tobias Burnus : https://gcc.gnu.org/g:f4e7ea81d1369d4d6cb6d8e440aefb3407142e05 commit r11-5571-gf4e7ea81d1369d4d6cb6d8e440aefb3407142e05 Author: Tobias Burnus Date:

[Bug fortran/98013] [OpenACC] 'gcc/fortran/trans-decl.c:gfc_generate_function_code' should consider 'flag_openacc' in addition to 'flag_openmp'?

2020-11-30 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98013 --- Comment #1 from CVS Commits --- The master branch has been updated by Tobias Burnus : https://gcc.gnu.org/g:f4e7ea81d1369d4d6cb6d8e440aefb3407142e05 commit r11-5571-gf4e7ea81d1369d4d6cb6d8e440aefb3407142e05 Author: Tobias Burnus Date:

[Patch, committed] Fortran's dump-parse-tree.c: Use '==' not '=' for '.eq.'.

2020-11-30 Thread Tobias Burnus
I was a bit confused by the '=' (assignment?) in the dump; hence, in line with /=, >= etc. it now is '==' for '==' or '.eq.'. Committed as r11-5570-g2610c786f7496c5006bb68d6801ef7450bd231a9 Tobias - Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany

Re: [PATCH] c++, debug: Treat -std=c++20 -gdwarf-5 like C++14 rather than C++98

2020-11-30 Thread Richard Biener via Gcc-patches
On Mon, Nov 30, 2020 at 3:12 PM Jakub Jelinek via Gcc-patches wrote: > > Hi! > > I have noticed that while we use DW_LANG_C_plus_plus_14 for -std=c++17 > -gdwarf-5, > we use DW_LANG_C_plus_plus (aka C++98) for -std=c++20 -gdwarf-5. The > following patch makes those two match. > > Ok for trunk

[Bug c++/98043] [8/9/10/11 Regression] ICE ‘verify_gimple’ failed since r5-3726-g083e891e69429f93b958f6c18e2d52f515bae572

2020-11-30 Thread mpolacek at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98043 Marek Polacek changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |mpolacek at gcc dot gnu.org

[Ada] Remove all ^L characters

2020-11-30 Thread Pierre-Marie de Rodat
These control characters are mainly causing confusion at this stage, so remove them. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * gcc-interface/Makefile.in, gcc-interface/trans.c: Remove ^L characters.diff --git a/gcc/ada/gcc-interface/Makefile.in

[Ada] Wrong replacement of Component.Discriminant

2020-11-30 Thread Pierre-Marie de Rodat
The procedure Replace_Discr_Ref added a few years ago is overzealous and triggers in too many cases in the case of multiple records with discriminants involved. It appears that this procedure is no longer needed at this stage, so simply remove it. Tested on x86_64-pc-linux-gnu, committed on trunk

[Ada] Fix internal error on extended return and fixed-point result

2020-11-30 Thread Pierre-Marie de Rodat
This prevents the expander from creating an access before elaboration issue for a temporary generated for a range check applied to the expression of a degenerate extended return statement (an extended return statement without an explicit do/end construct). Expand_N_Extended_Return_Statement has

[Ada] Implement inheritance for Default_Initial_Condition and address other gaps

2020-11-30 Thread Pierre-Marie de Rodat
The existing (SPARK-based) implementation of Default_Initial_Condition aspects only applies DIC checks based on a type's own DIC (or only one DIC from an ancestor if the type doesn't specify one, but not from further up the ancestor chain), but Ada 202x (AI12-0265) specifies that all DICs of

[Ada] Expand integer-only implementation of ordinary fixed-point types

2020-11-30 Thread Pierre-Marie de Rodat
This change relaxes the conditions under which the implementation of ordinary fixed-point types uses only underlying integer instructions. These conditions are on the Small of the ordinary fixed-point types, which must now be a rational number whose factors are of a magnitude bounded relatively

[Ada] Enable checks on runtime by default

2020-11-30 Thread Pierre-Marie de Rodat
These checks are not very costly these days and bring additional safety and security guarantees built into the Ada language, so enable them by default. It turns out that enabling checks on s-bitfie.adb makes a latent visibility bug appeared on strict alignment platform (related to alignment

[Ada] Reimplement Ada.Numerics.Big_Numbers.Big_Reals.Fixed_Conversions

2020-11-30 Thread Pierre-Marie de Rodat
This reimplements the aforementioned generic package according to the requirements of the Ada 2020 RM, namely that To_Big_Real be exact and that From_Big_Real use the common conversion rules. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * libgnat/a-nbnbre.adb

[Ada] Implement -gnateb switch

2020-11-30 Thread Pierre-Marie de Rodat
The -gnateb switch instructs gnat to store configuration pragma files by their basename in ALI files instead of using absolute paths. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * doc/gnat_ugn/building_executable_programs_with_gnat.rst: Describe -gnateb switch.

[Ada] Add stream-oriented attributes support for 128-bit integer types

2020-11-30 Thread Pierre-Marie de Rodat
This was overlooked in the original implementation of these types. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * doc/gnat_ugn/building_executable_programs_with_gnat.rst (-xdr): Document that XDR is not supported for 128-bit integer types. * gnat_ugn.texi:

[Ada] Compiler crash on limited conditional expressions

2020-11-30 Thread Pierre-Marie de Rodat
This patch fixes a bug in which if the expression of an expression function is a conditional expression of limited type, the compiler crashes. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * exp_ch3.adb (Expand_N_Object_Declaration): Avoid crash in case of

[Ada] Improve error recovery

2020-11-30 Thread Pierre-Marie de Rodat
Function P_Formal_Part was unnecessarily calling Scan after calling T_Semicolon: T_Semicolon is already calling Scan to go past the comma-used-insted-of-semicolon. This avoids spurious cascaded errors in case of e.g: procedure P (A : Integer, B : Integer, C : Integer) is where ',' is used

[Ada] Fix folding of comparison operators in GNATprove mode

2020-11-30 Thread Pierre-Marie de Rodat
Folding of comparison operators was disabled for GNATprove in assertion expressions (except for some special cases). However, folding itself is harmless. The problem was only in the current value optimizer, which interferes with assertions that act as cut points for proof. Tested on

[Ada] Fix serial port control setting on GNU/Linux

2020-11-30 Thread Pierre-Marie de Rodat
This fixes an issue when setting the control flags of the serial communication port. The c_cflag termios field should not be set with the baud rate. The speed of the communication is set using the c_ispeed and c_ospeed fields. Setting the baud rate into the c_cflag has unexpected results as it

[Ada] Spurious error on iterator over container with modified private part

2020-11-30 Thread Pierre-Marie de Rodat
The compiler rejects an element iterator (for .. of) over a container that extends a predefined container with additional operations in the private part, when these added operations overload existing GNAT-specific private operations used to optimize such loops. Tested on x86_64-pc-linux-gnu,

[Ada] Potential read of uninitialized variable in exp_dist.adb

2020-11-30 Thread Pierre-Marie de Rodat
Found by using -gnatVa and Initialize_Scalars on GNAT sources. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * exp_dist.adb (RCI_Cache): Initialize.diff --git a/gcc/ada/exp_dist.adb b/gcc/ada/exp_dist.adb --- a/gcc/ada/exp_dist.adb +++ b/gcc/ada/exp_dist.adb @@ -902,7

[Ada] Crash on ghost assignment check for illegal code

2020-11-30 Thread Pierre-Marie de Rodat
This patch fixes a bug, where an assignment of the form X(Y).Z := ... causes the compiler to crash when X does not refer to a visible declaration. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * errout.adb (Error_Msg_NEL): Do not call Set_Posted if errors are being

[Ada] Small cleanup in System.Value_F

2020-11-30 Thread Pierre-Marie de Rodat
This removes a superflous processing during the conversion to fixed point. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * libgnat/s-valuef.adb (Integer_To_Fixed): Do not modify numerator or denominator in order to reduce the exponent.diff --git

[Ada] Spurious visibility error in subprogram body in with_clause

2020-11-30 Thread Pierre-Marie de Rodat
Compiler rejects a use of a name in a subprogram body that appears in a with_clause, when the name is that of an inherited function for a local derived type, and is hidden by an explicit declaration of a non-overloadable homonym, and other homonyms exist in the environment of the subprogram body.

[Ada] Add continuation message when others choice not allowed

2020-11-30 Thread Pierre-Marie de Rodat
In a context where the bounds of an array aggregate are not known, the aggregate is not allowed to include an "others" choice. The error message given by GNAT now includes a suggestion to qualify the aggregate with a constrained subtype to fix the issue. Tested on x86_64-pc-linux-gnu, committed

[Ada] Confusion in Transform_Function_Array and internal subprograms

2020-11-30 Thread Pierre-Marie de Rodat
Generating an internal subprogram (e.g. array comparison functions) returning an array, the Transform_Function_Array mechanism gets confused and references the wrong entities in some cases. Fix this discrepency by directly transforming these functions into procedures instead of triggering the

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

2020-11-30 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729 --- Comment #24 from Richard Biener --- Amending / adjusting https://gcc.gnu.org/wiki/Building_Cross_Toolchains_with_gcc (the only place that somewhat "documents" how to setup AVR testing) is appreciated.

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

2020-11-30 Thread abebeos at lazaridis dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92729 --- Comment #23 from abebeos at lazaridis dot com --- (In reply to Senthil Kumar Selvaraj from comment #21) > (https://github.com/saaadhu/gcc-avr-cc0/tree/avr-cc0-squashed) I can still do a test-run, to see if it produces less fails than pip's

[Bug d/98067] [11 Regression] d: ICE in in force_decl_die, at dwarf2out.c:26197 with -gdwarf-2 -gstrict-dwarf

2020-11-30 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98067 --- Comment #7 from Jakub Jelinek --- (In reply to Richard Biener from comment #5) > > Would it be correct to fallback on a lang_hooks.name comparison if > > dwarf_version < 2? > > I wonder if we can instead "delay" applying "strict dwarf" to

Re: [00/23] Make fwprop use an on-the-side RTL SSA representation

2020-11-30 Thread Richard Sandiford via Gcc-patches
Jeff Law writes: > On 11/26/20 9:03 AM, Richard Sandiford wrote: >> Thanks for the reviews. >> >> Jeff Law via Gcc-patches writes: >>> On 11/13/20 1:10 AM, Richard Sandiford via Gcc-patches wrote: Just after GCC 10 stage 1 closed (oops), I posted a patch to add a new combine pass. One

[PATCH] c++, debug: Treat -std=c++20 -gdwarf-5 like C++14 rather than C++98

2020-11-30 Thread Jakub Jelinek via Gcc-patches
Hi! I have noticed that while we use DW_LANG_C_plus_plus_14 for -std=c++17 -gdwarf-5, we use DW_LANG_C_plus_plus (aka C++98) for -std=c++20 -gdwarf-5. The following patch makes those two match. Ok for trunk if it passes bootstrap/regtest? 2020-11-30 Jakub Jelinek * dwarf2out.c

[Bug lto/98068] [11 Regression] FAIL: gcc.dg/atomic/pr65345-4.c -O2 -flto -fno-use-linker-plugin -flto-partition=none (internal compiler error) by r11-3303

2020-11-30 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98068 --- Comment #2 from Martin Liška --- I guess it's PR97172.

[Bug jit/97867] [11 Regression] thunk_info::release breaks function calls in libgccjit

2020-11-30 Thread dmalcolm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97867 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug libstdc++/98005] FAIL: std/ranges/adaptors/sizeof.cc (test for excess errors)

2020-11-30 Thread ppalka at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98005 Patrick Palka changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug lto/98068] [11 Regression] FAIL: gcc.dg/atomic/pr65345-4.c -O2 -flto -fno-use-linker-plugin -flto-partition=none (internal compiler error) by r11-3303

2020-11-30 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98068 Richard Biener changed: What|Removed |Added Target Milestone|--- |11.0 --- Comment #1 from Richard

[PATCH] ipa: dump symtab to emergency dump file

2020-11-30 Thread Martin Liška
Hi. It's handy to have symbol table when we dump emergency dump. It's likely next stage1 material. Thoughts? Martin gcc/ChangeLog: * passes.c (emergency_dump_function): Dump symtab when we are in an IPA pass. --- gcc/passes.c | 3 +++ 1 file changed, 3 insertions(+) diff

<    1   2   3   >