Re: [PATCH] PR target/98152: Checking python is available before using

2020-12-07 Thread Kito Cheng via Gcc-patches
Committed On Tue, Dec 8, 2020 at 5:33 AM Jim Wilson wrote: > > On Sat, Dec 5, 2020 at 10:12 PM Kito Cheng wrote: > > > gcc/ChangeLog: > > > > * config.gcc (riscv*-*-*): Checking python, python3 or python2 > > is available, and skip doing with_arch canonicalize if no python > >

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

2020-12-07 Thread Richard Biener via Gcc-patches
On Mon, Dec 7, 2020 at 5:20 PM Qing Zhao wrote: > > > > On Dec 7, 2020, at 1:12 AM, Richard Biener wrote: > > On Fri, Dec 4, 2020 at 5:19 PM Qing Zhao wrote: > > > > > On Dec 4, 2020, at 2:50 AM, Richard Biener wrote: > > On Thu, Dec 3, 2020 at 6:33 PM Richard Sandiford > wrote: > > > Richard

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

2020-12-07 Thread Richard Biener via Gcc-patches
On Mon, Dec 7, 2020 at 7:34 PM Qing Zhao wrote: > > > > On Dec 7, 2020, at 12:05 PM, Richard Sandiford > wrote: > > Qing Zhao writes: > > On Dec 7, 2020, at 11:10 AM, Richard Sandiford > wrote: > > > Another issue is, in order to check whether an auto-variable has initializer, > I plan to

Re: [patch] Fix PR tree-optimization/96344

2020-12-07 Thread Richard Biener via Gcc-patches
On Mon, Dec 7, 2020 at 3:53 PM Eric Botcazou wrote: > > Hi, > > the very recent addition of the if_to_switch pass has partially disabled the > optimization I added back in June: > > 2020-06-26 Eric Botcazou > > * tree-ssa-reassoc.c (dump_range_entry): New function. >

Re: H8 cc0 conversion

2020-12-07 Thread Maciej W. Rozycki
On Sun, 22 Nov 2020, Jeff Law via Gcc-patches wrote: > This is a combination of my son's work as well as my own. Congratulations to Austin for this achievement! Maciej

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-07 Thread Julian Brown
Ping? Thanks, Julian On Fri, 13 Nov 2020 20:54:54 + Julian Brown wrote: > Hi Alexander, > > Thanks for the review! Comments below. > > On Tue, 10 Nov 2020 00:32:36 +0300 > Alexander Monakov wrote: > > > On Mon, 26 Oct 2020, Jakub Jelinek wrote: > > > > > On Mon, Oct 26, 2020 at

libgo patch commited: Don't use AF_LINK on Hurd

2020-12-07 Thread Ian Lance Taylor via Gcc-patches
This libgo patch by Svante Signell fixes the libgo build by not using AF_LINK on Hurd systems. This fixes GCC PR 98153. Bootstrapped and ran Go tests on x86_64-pc-linux-gnu. Committed to mainline. Ian eafa578e83e4d4f5e29e7e69e9701b4c55e3bd02 diff --git a/gcc/go/gofrontend/MERGE

[PATCH v4] rs6000, vector integer multiply/divide/modulo instructions

2020-12-07 Thread Carl Love via Gcc-patches
Will: I have addressed you comments with regards to the Change Log entries. The extra define vec_div was removed. Added the missing entries for DIVU_V2DI DIVS_V2DI in rs6000-call.c. The extra MULLD_V2DI case statement entry was removed. Added comment in rs6000.md about size for vector

[PATCH, powerpc] testsuite update tests for powerpc power10 target codegen.

2020-12-07 Thread will schmidt via Gcc-patches
[PATCH, powerpc] testsuite update tests for powerpc power10 target codegen. Hi, Assorted fix-ups to include prefixed load and store instructions in the scan-assembler stanzas for the gcc.target/powerpc tests. For these tests, we simply need to add pstxv or plxv added to the chain of

Re: [AArch64] Add --with-tune configure flag

2020-12-07 Thread Wilco Dijkstra via Gcc-patches
Hi Richard, >>> I share Richard E's concern about the effect of this on people who run >>> ./cc1 directly.  (And I'm being selfish here, because I regularly run >>> ./cc1 directly on toolchains configured with --with-arch=armv8.2-a+sve.) >>> So TBH my preference would be to keep the

Re: [C PATCH] fix atomic loads [PR 97981]

2020-12-07 Thread Joseph Myers
On Sat, 5 Dec 2020, Uecker, Martin wrote: > I should have taken the new warning for > > _Atomic int y; > y; // warning statement with no effect > > as a tell-tale sign that something is wrong, > although I still think the warning would be > correct. Or has a atomic load some special >

Re: Merge from trunk to gccgo branch

2020-12-07 Thread Ian Lance Taylor via Gcc-patches
I've now merged trunk revision b737b70fad398728f6006e8397d1bb31ccea4ce7 to the gccgo branch. Ian

Re: [PATCH] PR target/98152: Checking python is available before using

2020-12-07 Thread Jim Wilson
On Sat, Dec 5, 2020 at 10:12 PM Kito Cheng wrote: > gcc/ChangeLog: > > * config.gcc (riscv*-*-*): Checking python, python3 or python2 > is available, and skip doing with_arch canonicalize if no python > available. > Looks good to me. Jim

[PATCH][GCC] aarch64: Add +pauth to -march

2020-12-07 Thread Przemyslaw Wirkus via Gcc-patches
New +pauth (Pointer Authentication from Armv8.3-A) feature option for -march command line option. Please note that majority of PAUTH instructions are implemented behind HINT instruction. PAUTH stays a Armv8.3-A feature but now can be assigned to other architectures or CPUs. Patch includes: - new

Re: [PATCH,rs6000] Combine patterns for p10 load-cmpi fusion

2020-12-07 Thread will schmidt via Gcc-patches
On Fri, 2020-12-04 at 13:19 -0600, acsawdey--- via Gcc-patches wrote: > From: Aaron Sawdey > Assorted comments sprinkled around below. thanks -Will > This patch adds the first batch of patterns to support p10 fusion. These > will allow combine to create a single insn for a pair of

Re: [PATCH 1/2] libstdc++: Add --enable-pure-stdio-libstdcxx option

2020-12-07 Thread Keith Packard via Gcc-patches
Jonathan Wakely writes: > GCC changelog files are autogenerated now, so patches should not touch > them. Just include the ChangeLog entry in the Git commit log (which > will usually end up being quoted in the patch and/or the email body of > the mail to gcc-patches). Awesome. > I think the

Re: [AArch64] Add --with-tune configure flag

2020-12-07 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra via Gcc-patches writes: > Hi Richard, > >> I share Richard E's concern about the effect of this on people who run >> ./cc1 directly.  (And I'm being selfish here, because I regularly run >> ./cc1 directly on toolchains configured with --with-arch=armv8.2-a+sve.) >> So TBH my

Re: [PATCH] Remove misleading debug line entries

2020-12-07 Thread Bernd Edlinger
On 12/7/20 11:50 AM, Richard Biener wrote: > The ipa-param-manipulation.c hunk is OK, please commit separately. > done. > The tree-inline.c and cfgexpand.c changes are OK as well, for the > tree-ssa-live.c change see below > > > From 85b0e37d0c0d3ecac4908ebbfd67edc612ef22b2 Mon Sep 17

Re: [PATCH 1/2] libstdc++: Add --enable-pure-stdio-libstdcxx option

2020-12-07 Thread Jonathan Wakely via Gcc-patches
On 07/12/20 10:39 -0800, Keith Packard via Libstdc++ wrote: This option directs the library to only use simple stdio APIs instead of using fileno to get the file descriptor for use with POSIX APIs. Makes sense, thanks. I've replaced the gcc@ list with gcc-patches@ in the recipients.

Re: [AArch64] Add --with-tune configure flag

2020-12-07 Thread Wilco Dijkstra via Gcc-patches
Hi Richard, > I share Richard E's concern about the effect of this on people who run > ./cc1 directly.  (And I'm being selfish here, because I regularly run > ./cc1 directly on toolchains configured with --with-arch=armv8.2-a+sve.) > So TBH my preference would be to keep the

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

2020-12-07 Thread Qing Zhao via Gcc-patches
> On Dec 7, 2020, at 12:05 PM, Richard Sandiford > wrote: > > Qing Zhao mailto:qing.z...@oracle.com>> writes: >>> On Dec 7, 2020, at 11:10 AM, Richard Sandiford >>> wrote: >> >> Another issue is, in order to check whether an auto-variable has >> initializer, I plan to add a

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

2020-12-07 Thread Richard Sandiford via Gcc-patches
Qing Zhao writes: >> On Dec 7, 2020, at 11:10 AM, Richard Sandiford >> wrote: > > Another issue is, in order to check whether an auto-variable has > initializer, I plan to add a new bit in “decl_common” as: > /* In a VAR_DECL, this is DECL_IS_INITIALIZED. */ > unsigned

Go patch committed: The type of a string index is byte

2020-12-07 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend sets the type of a string index expression to byte, which is a type alias. To make this work from the do_type method, we add "byte" and "rune" to the list of known integer types, and look them up that way rather than via gogo->lookup_global. This is for

Go testsuite patch committed: Don't quote quoted parentheses

2020-12-07 Thread Ian Lance Taylor via Gcc-patches
This patch fixes go-test.exp to not backslash quote parentheses that are already backslash quoted. This regexp quoting isn't fully general, it just has to handle the cases that arise in the testsuite. This change fixes a case in a new test, not yet committed. (It would be nice to have a general

Re: [PATCH] builtins: Avoid ICE with __builtin_clear_padding on POINTERS_EXTEND_UNSIGNED targets [PR98147]

2020-12-07 Thread Richard Sandiford via Gcc-patches
Jakub Jelinek via Gcc-patches writes: > Hi! > > The function that calls targetm.emit_call_builtin___clear_cache > asserts that each of the begin and end operands has either ptr_mode or > Pmode. > On most targets that is the same mode, but e.g. on aarch64 -mabi=ilp32 > or a few others it is

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

2020-12-07 Thread Qing Zhao via Gcc-patches
> On Dec 7, 2020, at 11:10 AM, Richard Sandiford > wrote: Another issue is, in order to check whether an auto-variable has initializer, I plan to add a new bit in “decl_common” as: /* In a VAR_DECL, this is DECL_IS_INITIALIZED. */ unsigned decl_is_initialized :1;

Go patch committed: Don't name type descriptor for alias type

2020-12-07 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend avoids using an alias type name when naming a type descriptor. The test case for this https://golang.org/cl/275632. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian 47cf6054ce8d11aa681846d08433afdb4404abe7 diff --git

Re: [AArch64] Add --with-tune configure flag

2020-12-07 Thread Richard Sandiford via Gcc-patches
"Pop, Sebastian" writes: > On 11/19/20, 10:52 AM, "Richard Earnshaw (lists)" > wrote: >> Having the same option have a completely different meaning would be even >> worse than not having the option at all. So no, that's a non-starter. > > The attached patch 0001 removes

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

2020-12-07 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > On Thu, Dec 3, 2020 at 6:33 PM Richard Sandiford > wrote: >> >> Richard Biener via Gcc-patches writes: >> > On Tue, Nov 24, 2020 at 4:47 PM Qing Zhao wrote: >> >> Another issue is, in order to check whether an auto-variable has >> >> initializer, I plan to add a new

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

2020-12-07 Thread Richard Sandiford via Gcc-patches
Qing Zhao via Gcc-patches writes: >> On Dec 7, 2020, at 1:12 AM, Richard Biener >> wrote: >> >> On Fri, Dec 4, 2020 at 5:19 PM Qing Zhao > > wrote: >>> >>> >>> >>> On Dec 4, 2020, at 2:50 AM, Richard Biener >>> wrote: >>> >>> On Thu, Dec 3, 2020 at 6:33 PM

Re: [PATCH] Overflow-trapping integer arithmetic routines7code

2020-12-07 Thread Stefan Kanthak
Jeff Law wrote Wednesday, November 25, 2020 7:11 PM: > On 11/25/20 6:18 AM, Stefan Kanthak wrote: >> Jeff Law wrote: [...] >>> My inclination is to leave the overflow checking double-word multiplier >>> as-is. >> See but ff. >

c++: Adjust array type construction

2020-12-07 Thread Nathan Sidwell
This restores the dependent array changes I reverted, now that pr98116 appears fixed. As mentioned before, when deserializing a module we need to construct arrays without using the dependent-type predicates themselves. gcc/cp/ * cp-tree.h (build_cplus_array_type): Add defaulted

c++: check alias match for specializations [PR98116]

2020-12-07 Thread Nathan Sidwell
This fixes the underlying problem my recent (backedout) changesto array type creation uncovered.We had paths through structural_comptypes that ignored alias templates, even when significant. This addsthe necessary checks. PR c++/98116 gcc/cp/ * typeck.c

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

2020-12-07 Thread Qing Zhao via Gcc-patches
> On Dec 7, 2020, at 1:12 AM, Richard Biener wrote: > > On Fri, Dec 4, 2020 at 5:19 PM Qing Zhao > wrote: >> >> >> >> On Dec 4, 2020, at 2:50 AM, Richard Biener >> wrote: >> >> On Thu, Dec 3, 2020 at 6:33 PM Richard Sandiford >> wrote: >> >> >> Richard

Re: [committed] Fix non-unique testnames

2020-12-07 Thread Martin Sebor via Gcc-patches
On 12/4/20 3:17 PM, Jeff Law via Gcc-patches wrote: On 12/4/20 2:55 PM, Mike Stump wrote: On Nov 30, 2020, at 8:00 AM, Jeff Law via Gcc-patches wrote: 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

Re: [PATCH] Avoid atomic for guard acquire when that is expensive

2020-12-07 Thread Bernd Edlinger
On 12/7/20 4:04 PM, Jason Merrill wrote: > On 12/5/20 7:37 AM, Bernd Edlinger wrote: >> On 12/2/20 7:57 PM, Jason Merrill wrote: >>> On 12/1/20 1:28 PM, Bernd Edlinger wrote: +  tree type = >>> targetm.cxx.guard_mask_bit () +  ? TREE_TYPE (guard) : char_type_node; +

Re: [PATCH] i386: Add combine splitters to allow combining multiple insns into reg1 = const; reg2 = rotate (reg1, reg3 & cst) [PR96226]

2020-12-07 Thread Uros Bizjak via Gcc-patches
On Mon, Dec 7, 2020 at 4:30 PM Segher Boessenkool wrote: > > Hi! > > On Mon, Dec 07, 2020 at 03:27:14PM +0100, Uros Bizjak wrote: > > On Fri, Dec 4, 2020 at 7:26 PM Segher Boessenkool > > wrote: > > > A splitter can *already* split to only one insn. > > > > Unfortunately, the attached patch with

Re: [PATCH] i386: Add combine splitters to allow combining multiple insns into reg1 = const; reg2 = rotate (reg1, reg3 & cst) [PR96226]

2020-12-07 Thread Segher Boessenkool
On Mon, Dec 07, 2020 at 04:40:39PM +0100, Jakub Jelinek wrote: > On Mon, Dec 07, 2020 at 09:28:19AM -0600, Segher Boessenkool wrote: > > I see. We could also allow this for 2->2 combinations (w/ the same > > restrictions we do for other 2->2 combinations, but that probably falls > > out

Re: [PATCH] i386: Add combine splitters to allow combining multiple insns into reg1 = const; reg2 = rotate (reg1, reg3 & cst) [PR96226]

2020-12-07 Thread Jakub Jelinek via Gcc-patches
On Mon, Dec 07, 2020 at 09:28:19AM -0600, Segher Boessenkool wrote: > I see. We could also allow this for 2->2 combinations (w/ the same > restrictions we do for other 2->2 combinations, but that probably falls > out automatically). In this case we actually want a 2->1 combination, where the

[COMMITTED] Fix location info in ipa_param_body_adjustments::modify_call_stmt

2020-12-07 Thread Bernd Edlinger
On 12/7/20 11:50 AM, Richard Biener wrote: > > The ipa-param-manipulation.c hunk is OK, please commit separately. > Okay, this is the part I just committed. commit 92e563d91b012f09da8fd152e934f6b964ae49cb Author: Bernd Edlinger Date: Mon Dec 7 16:00:00 2020 +0100 Fix location info in

Re: [PATCH] i386: Add combine splitters to allow combining multiple insns into reg1 = const; reg2 = rotate (reg1, reg3 & cst) [PR96226]

2020-12-07 Thread Segher Boessenkool
Hi! On Mon, Dec 07, 2020 at 03:27:14PM +0100, Uros Bizjak wrote: > On Fri, Dec 4, 2020 at 7:26 PM Segher Boessenkool > wrote: > > A splitter can *already* split to only one insn. > > Unfortunately, the attached patch with the following testcase: > > --cut here-- > int test (int a, int b) > { >

Re: [PATCH] Avoid atomic for guard acquire when that is expensive

2020-12-07 Thread Jason Merrill via Gcc-patches
On 12/5/20 7:37 AM, Bernd Edlinger wrote: On 12/2/20 7:57 PM, Jason Merrill wrote: On 12/1/20 1:28 PM, Bernd Edlinger wrote: On 11/24/20 11:10 PM, Jason Merrill wrote: On 11/22/20 3:05 AM, Bernd Edlinger wrote: Hi, this avoids the need to use -fno-threadsafe-statics on arm-none-eabi or

[committed][wwwdocs] gcc-11/changes: "used" attribute saves decls from linker garbage collection

2020-12-07 Thread Jozef Lawrynowicz
Since 6fbec038f7a, the "used" attribute will save the symbol declaration it is applied to from linker garbage collection, if the target supports the SHF_GNU_RETAIN section flag. This patch documents this functionality in the GCC 11 changes document. Some users might need to amend their source

Re: [PATCH] c-family: Fix hang with -Wsequence-point [PR98126]

2020-12-07 Thread Jason Merrill via Gcc-patches
On 12/4/20 10:40 PM, Marek Polacek wrote: verify_sequence_points uses verify_tree to recursively walk the subexpressions of an expression, and while recursing, it also keeps lists of expressions found after/before a sequence point. For a large expression, the list can grow significantly. And

Re: [PATCH v2] c++: ICE with switch and scoped enum bit-fields [PR98043]

2020-12-07 Thread Jason Merrill via Gcc-patches
On 12/4/20 10:46 PM, Marek Polacek wrote: On Wed, Dec 02, 2020 at 09:50:33PM -0500, Jason Merrill wrote: On 12/2/20 6:18 PM, Marek Polacek wrote: In this testcase we are crashing trying to gimplify a switch, because the types of the switch condition and case constants have different

[patch] Fix PR tree-optimization/96344

2020-12-07 Thread Eric Botcazou
Hi, the very recent addition of the if_to_switch pass has partially disabled the optimization I added back in June: 2020-06-26 Eric Botcazou * tree-ssa-reassoc.c (dump_range_entry): New function. (debug_range_entry): New debug function. (update_range_test): Invoke

[committed] doc: "used" attribute saves decls from linker garbage collection

2020-12-07 Thread Jozef Lawrynowicz
Since 6fbec038f7a, the "used" attribute will save the symbol declaration it is applied to from linker garbage collection, if the target supports the SHF_GNU_RETAIN section flag. This patch documents this new functionality. Committed as obvious. >From 724390745213d5192af04a51bb08cf44da7c396d Mon

Re: [PATCH] i386: Add combine splitters to allow combining multiple insns into reg1 = const; reg2 = rotate (reg1, reg3 & cst) [PR96226]

2020-12-07 Thread Uros Bizjak via Gcc-patches
On Fri, Dec 4, 2020 at 7:26 PM Segher Boessenkool wrote: > > Hi! > > On Fri, Dec 04, 2020 at 07:06:45PM +0100, Uros Bizjak wrote: > > On Fri, Dec 4, 2020 at 6:57 PM Jakub Jelinek wrote: > > > > > > On Fri, Dec 04, 2020 at 06:53:49PM +0100, Uros Bizjak wrote: > > > > > > I was trying that first,

Re: [PATCH 00/31] VAX: Bring the port up to date (yes, MODE_CC conversion is included)

2020-12-07 Thread Maciej W. Rozycki
On Sun, 29 Nov 2020, Martin Sebor wrote: > > Perhaps it has been fixed since, but mentioning it in case it has not. > > I wouldn't expect to see this warning after r11-5073. If it persists, > can you please open a bug and attach the translation unit to it? Indeed, the issue has gone now, now

RE: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for vmvn intrinsics

2020-12-07 Thread Kyrylo Tkachov via Gcc-patches
> -Original Message- > From: Prathamesh Kulkarni > Sent: 07 December 2020 11:26 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for > vmvn intrinsics > > On Mon, 7 Dec 2020 at 16:34, Kyrylo Tkachov > wrote: >

Re: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for vmvn intrinsics

2020-12-07 Thread Prathamesh Kulkarni via Gcc-patches
On Mon, 7 Dec 2020 at 16:34, Kyrylo Tkachov wrote: > > Hi Prathamesh, > > > -Original Message- > > From: Prathamesh Kulkarni > > Sent: 07 December 2020 11:01 > > To: Kyrylo Tkachov > > Cc: gcc-patches@gcc.gnu.org > > Subject: Re: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~

[PATCH] builtins: Avoid ICE with __builtin_clear_padding on POINTERS_EXTEND_UNSIGNED targets [PR98147]

2020-12-07 Thread Jakub Jelinek via Gcc-patches
Hi! The function that calls targetm.emit_call_builtin___clear_cache asserts that each of the begin and end operands has either ptr_mode or Pmode. On most targets that is the same mode, but e.g. on aarch64 -mabi=ilp32 or a few others it is different. When a target has a clear cache non-library

RE: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for vmvn intrinsics

2020-12-07 Thread Kyrylo Tkachov via Gcc-patches
Hi Prathamesh, > -Original Message- > From: Prathamesh Kulkarni > Sent: 07 December 2020 11:01 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for > vmvn intrinsics > > On Thu, 3 Dec 2020 at 16:05, Kyrylo

Re: [PATCH] Optimize macro: make it more predictable

2020-12-07 Thread Martin Liška
PING^2 On 11/26/20 2:56 PM, Martin Liška wrote: PING^1 On 11/9/20 11:35 AM, Martin Liška wrote: On 11/3/20 2:34 PM, Jakub Jelinek wrote: On Tue, Nov 03, 2020 at 02:27:52PM +0100, Richard Biener wrote: On Fri, Oct 23, 2020 at 1:47 PM Martin Liška wrote: This is a follow-up of the

Re: [PR66791][ARM] Replace calls to __builtin_neon_vmvn* by ~ for vmvn intrinsics

2020-12-07 Thread Prathamesh Kulkarni via Gcc-patches
On Thu, 3 Dec 2020 at 16:05, Kyrylo Tkachov wrote: > > > > > -Original Message- > > From: Gcc-patches On Behalf Of > > Prathamesh Kulkarni via Gcc-patches > > Sent: 03 December 2020 10:30 > > To: gcc Patches ; Kyrill Tkachov > > > > Subject: Re: [PR66791][ARM] Replace calls to

Re: [PATCH] d: Add darwin support for D language front-end

2020-12-07 Thread Iain Buclaw via Gcc-patches
Hi, Sorry it's been delayed by a bit. Currently Iain (not that Iain) is running tests on a wide net of versions and architectures, with mixed success rates that seem to boil down to some C binding issue. Most of the published effort on my side is currently sitting in users/ibuclaw/darwin.

Re: [PATCH] Remove misleading debug line entries

2020-12-07 Thread Richard Biener
On Fri, 4 Dec 2020, Bernd Edlinger wrote: > On 12/3/20 9:30 AM, Richard Biener wrote: > > On Wed, 2 Dec 2020, Bernd Edlinger wrote: > > > >> On 12/2/20 8:50 AM, Richard Biener wrote: > >>> On Tue, 1 Dec 2020, Bernd Edlinger wrote: > >>> > Hi! > > > This removes gimple_debug

Re: [PATCH] tree-optimization/98117 - fix range set by vectorization on niter IVs

2020-12-07 Thread Richard Biener
On Mon, 7 Dec 2020, Richard Sandiford wrote: > Richard Biener writes: > > This avoids the degenerate case of a TYPE_MAX_VALUE latch iteration > > count value causing wrong range info for the vector IV. There's > > still the case of VF == 1 where if we don't know whether we hit the > > above

Re: [PATCH] tree-optimization/98117 - fix range set by vectorization on niter IVs

2020-12-07 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > This avoids the degenerate case of a TYPE_MAX_VALUE latch iteration > count value causing wrong range info for the vector IV. There's > still the case of VF == 1 where if we don't know whether we hit the > above case we cannot emit a range. > > Bootstrap and regtest

[Ada] Fix internal error on library-level type extended locally

2020-12-07 Thread Eric Botcazou
This is a regression present on the mainline, 10 and 9 branches: the compiler aborts on the local extension of a tagged type declared at library level, with a progenitor given by an interface type having a primitive that is a homograph of a primitive of the tagged type. Tested on x86-64/Linux,

[Ada] Fix assembler name collision

2020-12-07 Thread Eric Botcazou
Gigi uses a dummy global variable to register global types for debug info purposes and its name can now collide with user variables. Tested on x86-64/Linux, applied on the mainline and 10 branch. 2020-12-07 Eric Botcazou * gcc-interface/trans.c (lvalue_for_aggregate_p): Also return

[PATCH] tree-optimization/98117 - fix range set by vectorization on niter IVs

2020-12-07 Thread Richard Biener
This avoids the degenerate case of a TYPE_MAX_VALUE latch iteration count value causing wrong range info for the vector IV. There's still the case of VF == 1 where if we don't know whether we hit the above case we cannot emit a range. Bootstrap and regtest running on x86_64-unknown-linux-gnu.

Re: [patch] Enhance debug info for fixed-point types

2020-12-07 Thread Eric Botcazou
> 2020-11-11 Eric Botcazou > > * exp_dbug.adb (Is_Handled_Scale_Factor): Delete. > (Get_Encoded_Name): Do not call it. > * gcc-interface/decl.c (gnat_to_gnu_entity) : > Tidy up and always use a meaningful description for arbitrary > scale factors. > *

[Ada] Fix corner case issue with discriminated record type

2020-12-07 Thread Eric Botcazou
The compiler generates code that writes too much data into a component of a record subject to a representation clause, when the source of the assignment is a call to a function that returns a discriminated record type with default discriminants, variable size and a statically known upper bound

[Ada] Fix problematic conversion to boolean type

2020-12-07 Thread Eric Botcazou
The new ranger exposed a problematic conversion to boolean type done in gigi. Tested on x86-64/Linux, applied on the mainline. 2020-12-07 Eric Botcazou * gcc-interface/utils.c (convert) : Call fold_convert in the cases where convert_to_integer is not called. : Call

Re: [Ada] Build support units for 128-bit integer types on 64-bit platforms

2020-12-07 Thread Maciej W. Rozycki
On Thu, 3 Dec 2020, Andreas Schwab wrote: > > .../gcc/gnatbind -x impbit.ali > > error: "s-imgllli.ali" not found, "s-imgllli.ads" must be compiled > > gnatmake: *** bind failed. > > Failed to compile impbit > > This means GNATRTL_128BIT_OBJS is missing from > EXTRA_GNATRTL_NONTASKING_OBJS.

[Committed] IBM Z: Change Pmode to word_mode for stack probes

2020-12-07 Thread Andreas Krebbel via Gcc-patches
In s390.c we are still using Pmode for the stack probes. This breaks with -m31 -mzarch where Pmode != word_mode. The patch also adds a new target check to s390.exp which allows us to implement zarch specific checks in the testcases. Bootstrapped and regression tested on s390x with and without

Re: [patch] [ada] Fix PR ada/97504 for mips*-linux

2020-12-07 Thread Arnaud Charlet
> Fix PR ada/97504 for mips*-linux, the bootstrap works again on mips*-linux. > Ok for the trunk? OK, thanks. > gcc/ada/ > PR ada/97504 > * Makefile.rtl (LIBGNAT_TARGET_PAIRS) : Use wraplf > version of Aux_Long_Long_Float. > > --- a/gcc/ada/Makefile.rtl > +++

[patch] [ada] Fix PR ada/97504 for mips*-linux

2020-12-07 Thread Matthias Klose
Fix PR ada/97504 for mips*-linux, the bootstrap works again on mips*-linux. Ok for the trunk? gcc/ada/ PR ada/97504 * Makefile.rtl (LIBGNAT_TARGET_PAIRS) : Use wraplf version of Aux_Long_Long_Float. --- a/gcc/ada/Makefile.rtl +++ b/gcc/ada/Makefile.rtl @@ -2288,6 +2288,7

Re: Updating the backend status for h8300 on the wiki

2020-12-07 Thread Eric Botcazou
> I noticed that as well. OK, now corrected. -- Eric Botcazou

Re: Updating the backend status for h8300 on the wiki

2020-12-07 Thread John Paul Adrian Glaubitz
On 12/7/20 9:06 AM, Eric Botcazou wrote: >> Now there are only AVR and CR16 that need to be converted. Great progress! > > Indeed, but why does CR16 not have the 'c' letter then? I noticed that as well. Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer - glaub...@debian.org

Re: [patch] don't build insn-extract.o with rtl checking

2020-12-07 Thread Richard Biener
On Mon, 7 Dec 2020, Matthias Klose wrote: > As seen in PR98144, building insn-extract.o with rtl checking takes some > memory, > and it doesn't work on 32bit architectures at all (PR97314). Richard > suggested > on irc to disable rtl checking for this auto-generated file, like it's already >

[patch] don't build insn-extract.o with rtl checking

2020-12-07 Thread Matthias Klose
As seen in PR98144, building insn-extract.o with rtl checking takes some memory, and it doesn't work on 32bit architectures at all (PR97314). Richard suggested on irc to disable rtl checking for this auto-generated file, like it's already done for genconditions.c. Patching it like done for

Re: Updating the backend status for h8300 on the wiki

2020-12-07 Thread Eric Botcazou
> Now there are only AVR and CR16 that need to be converted. Great progress! Indeed, but why does CR16 not have the 'c' letter then? -- Eric Botcazou