Re: [PATCH 00/29] rs6000: Auto-generate builtins from descriptions [V2]

2020-07-27 Thread Bill Schmidt via Gcc-patches
Just a reminder this patch series exists and wants a review. :-) Bill On 7/27/20 9:13 AM, Bill Schmidt wrote: From: Bill Schmidt This is a slight reworking of the patches posted on June 17. I have made a couple of improvements, but the general arrangement of the patches is the same as before

Re: [PATCH 8/9] [OpenACC] Fix standalone attach for Fortran assumed-shape array pointers

2020-07-27 Thread Julian Brown
On Fri, 17 Jul 2020 13:16:11 +0200 Thomas Schwinge wrote: > Hi Julian, Tobias! > > On 2020-07-15T12:28:42+0200, Thomas Schwinge > wrote: > > On 2020-07-14T13:43:37+0200, I wrote: > >> On 2020-06-16T15:39:44-0700, Julian Brown > >> wrote: > >>> As mentioned in the blurb for the previous pat

Fwd: [PATCH 00/29] rs6000: Auto-generate builtins from descriptions [V2]

2020-07-27 Thread Bill Schmidt via Gcc-patches
I apologize for the useless "From" address (and lack of "Reply-To" address) on this patch series.  My usual machine is down for maintenance, so I ended up sending this from my laptop, which was clearly not configured well enough for that.  My bad.  I won't do that again. Meantime, please repl

[PATCH] config/debuginfod.m4: Restore AC_CHECK_LIB check

2020-07-27 Thread H.J. Lu via Gcc-patches
On Sat, Jul 25, 2020 at 9:01 AM H.J. Lu wrote: > > On Fri, Jul 24, 2020 at 1:04 PM Aaron Merey via Gcc-patches > wrote: > > > > On Tue, Jul 21, 2020 at 2:11 PM Aaron Merey wrote: > > > > > > On Tue, Jul 21, 2020 at 11:20 AM Tom Tromey wrote: > > > > > > > > Simon> Since it's debuginfo.m4 that i

Re: [PATCH] expr: build string_constant only for a char type

2020-07-27 Thread Martin Sebor via Gcc-patches
On 7/27/20 6:32 AM, Martin Liška wrote: Hey. As mentioned in the PR, we should not create a string constant for a type that is different from char_type_node. Looking at expr.c, I was inspired and used 'TYPE_MAIN_VARIANT (chartype) == char_type_node' to verify that underlying type is a characte

Re: [PATCH] config/debuginfod.m4: Restore AC_CHECK_LIB check

2020-07-27 Thread Aaron Merey via Gcc-patches
On Mon, Jul 27, 2020 at 11:32 AM H.J. Lu wrote: > > On Sat, Jul 25, 2020 at 9:01 AM H.J. Lu wrote: > > This caused: > > > > https://sourceware.org/bugzilla/show_bug.cgi?id=26301 > > > > It is quite normal to have debuginfod headers without libdebuginfod on > multilib OSes. Restore AC_CHECK_LIB t

Re: [PATCH] libgomp: Add helper functions for memory handling.

2020-07-27 Thread Jakub Jelinek via Gcc-patches
On Sat, Jul 25, 2020 at 11:03:27AM -0400, y2s1982 via Gcc-patches wrote: > This patch adds few helper functions aims to improve readability of > fetching addresses, sizes, and values. It also proposes a syntax for > querying these information through the callback functions, similar to > that of LLV

Go patch committed: Scan all function literals for escape analysis

2020-07-27 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend scans all function literals for escape analysis. We were scanning only function literals with closures, but not all function literals have closures. The effect of this is a missed optimization in some cases: we will allocate some variables on the heap unnecessarily.

Re: [PATCH v4] driver: fix a problem with implementation of -falign-foo=0 [PR96247]

2020-07-27 Thread Richard Sandiford
Hu Jiangping writes: > Hi! > > This patch makes the -falign-foo=0 work as described in the > documentation. Thanks for all the suggestions. > > v4: do changes for coding conventions > v3: make change more readable and self-consistent > > Changelog: > 2020-07-27 Hu Jiangping > > PR driver/

Re: [PATCH PR95696] regrename creates overlapping register allocations for vliw

2020-07-27 Thread Richard Sandiford
Zhongyunde writes: > I reconsider the issue and update patch attached. > Yes, If the kernel loop bb's information doesn't use in regrename, it > also need not be collected to save compile time. Thanks. The point about other callers was a good one though. I think it would be OK to add a new param

Re: [PATCH] libgomp: Add helper functions for memory handling.

2020-07-27 Thread y2s1982 via Gcc-patches
Hello Jakub, Thanks for the reply. I apparently need further clarification. On Mon, Jul 27, 2020 at 12:36 PM Jakub Jelinek wrote: > On Sat, Jul 25, 2020 at 11:03:27AM -0400, y2s1982 via Gcc-patches wrote: > > This patch adds few helper functions aims to improve readability of > > fetching addre

Re: [PATCH] expr: build string_constant only for a char type

2020-07-27 Thread Martin Liška
On 7/27/20 5:53 PM, Martin Sebor wrote: The tests I committed with the change didn't exercise any of this so that's my bad.  I'm still not sure I understand how the problem with the incomplete type comes up (I haven't had a chance to look into the recent updates on the bug yet) but to retain the 

[PATCH] PKG_CHECK_MODULES: Check if $pkg_cv_[]$1[]_LIBS works

2020-07-27 Thread H.J. Lu via Gcc-patches
On Mon, Jul 27, 2020 at 9:11 AM Aaron Merey wrote: > > On Mon, Jul 27, 2020 at 11:32 AM H.J. Lu wrote: > > > > On Sat, Jul 25, 2020 at 9:01 AM H.J. Lu wrote: > > > This caused: > > > > > > https://sourceware.org/bugzilla/show_bug.cgi?id=26301 > > > > > > > It is quite normal to have debuginfod h

V2 [PATCH] PKG_CHECK_MODULES: Check if $pkg_cv_[]$1[]_LIBS works

2020-07-27 Thread H.J. Lu via Gcc-patches
On Mon, Jul 27, 2020 at 12:14 PM H.J. Lu wrote: > > On Mon, Jul 27, 2020 at 9:11 AM Aaron Merey wrote: > > > > On Mon, Jul 27, 2020 at 11:32 AM H.J. Lu wrote: > > > > > > On Sat, Jul 25, 2020 at 9:01 AM H.J. Lu wrote: > > > > This caused: > > > > > > > > https://sourceware.org/bugzilla/show_bug

Re: [PATCH] expr: build string_constant only for a char type

2020-07-27 Thread Martin Sebor via Gcc-patches
On 7/27/20 12:54 PM, Martin Liška wrote: On 7/27/20 5:53 PM, Martin Sebor wrote: The tests I committed with the change didn't exercise any of this so that's my bad.  I'm still not sure I understand how the problem with the incomplete type comes up (I haven't had a chance to look into the recent 

c-family: Use strcmp to compare location file names

2020-07-27 Thread Nathan Sidwell
The logic to figure out where a missing #include should be inserted uses pointer equality to check filenames -- the routine even says so. But cpplib makes no such guarantee. It happens to be true for input that it preprocesses[* see line zero below], but is not true for source that has already

Re: [PATCH] expr: build string_constant only for a char type

2020-07-27 Thread Jakub Jelinek via Gcc-patches
On Mon, Jul 27, 2020 at 09:53:31AM -0600, Martin Sebor via Gcc-patches wrote: > Return a pointer P to a NUL-terminated string containing > the sequence of bytes corresponding to the representation > of the object referred to by SRC (or a subsequence of such > bytes within it if SRC is a ref

[PATCH v2] ipa/96291: don't crash on unoptimized lto functions

2020-07-27 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich In PR ipa/96291 the test contained an SCC with one unoptimized function. This tricked ipa-cp into NULL dereference. has_undead_caller_from_outside_scc_p() did not take into account that unoptimized funtions don't have IPA summary analysis. And dereferenced NULL pointer c

Re: [PATCH] ipa/96291: don't crash on unoptimized lto functions

2020-07-27 Thread Sergei Trofimovich via Gcc-patches
On Mon, 27 Jul 2020 14:41:14 +0200 Martin Jambor wrote: > Hi, > > On Mon, Jul 27 2020, Richard Biener via Gcc-patches wrote: > > On Sat, Jul 25, 2020 at 8:35 PM Sergei Trofimovich via Gcc-patches > > wrote: > >> > >> From: Sergei Trofimovich > >> > >> In PR ipa/96291 the test contained an SC

Re: [PATCH] RISC-V: Add ZFINX support

2020-07-27 Thread Jim Wilson
On Sun, Jul 26, 2020 at 11:40 PM wangtao (CH) wrote: > This is the patch to support ZFINX of RISC-V, which option is like > -march=rv32gc_zfinx. The ZFINX means f-registers in x-registers under RV-F > and RV-D extension. For more details, please refer to > https://github.com/riscv/riscv-zfinx/b

Re: [PATCH v2] [RISC-V] Add support for TLS stack protector canary access

2020-07-27 Thread Jim Wilson
On Sun, Jul 19, 2020 at 7:04 PM cooper wrote: > Ping > > On 2020/7/13 下午4:15, cooper wrote: > > gcc/ > > * config/riscv/riscv-opts.h (stack_protector_guard): New enum. > > * config/riscv/riscv.c (riscv_option_override): Handle > > the new options. > > * config/riscv/riscv.m

Re: [PATCH] aarch64: Delete duplicated option docs.

2020-07-27 Thread Jim Wilson
Ping. ccing the aarch64 maintainers. If I don't get a response, I will just commit this as obvious. Jim On Sun, Jul 12, 2020 at 4:52 PM Jim Wilson wrote: > > Noticed while reviewing the RISC-V -mstack-protector-guard docs. The, and > could maybe be added as a separate patch. > AArch64 secti

gcc.dg/torture/pr39074-2.c, pr39074.c, pta-callused-1.c: Adjust for mmix.

2020-07-27 Thread Hans-Peter Nilsson
Committed. These FAILs for mmix showed up as regressions for me due to a flaw in the btest-gcc.sh test-results-accounting: a bug was recently fixed regarding the naming of dump-files so the names are again correct. To wit, parts of the tests that were UNRESOLVED, due to missing dump-files, and ig

Go patch committed: turn global "a = b; b = x' to "a = x" when possible

2020-07-27 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend changes package-scope "a = b; b = x" to just set "a = x". This avoids requiring an init function to initialize the variable. This can only be done if x is a static initializer. The go1.15rc1 runtime package relies on this optimization. The package has a variable "v

Go patch committed: Pass only ptr and len to some runtime calls

2020-07-27 Thread Ian Lance Taylor via Gcc-patches
This patch changes the Go frontend and the libgo runtime package to pass only ptr and len to some runtime calls, rather than passing an entire slice. This ports https://golang.org/cl/227163 to the Go frontend. This is a step toward moving up to the go1.15rc1 release. The original change says "So

Re: [PATCH] MIPS: Fix __builtin_longjmp (PR 64242)

2020-07-27 Thread Paul Hua via Gcc-patches
ping? On Sun, Jul 12, 2020 at 2:27 PM Paul Hua wrote: > > From 589dbe8a1c2397bfafefa4e84abe5ec6e6798928 Mon Sep 17 00:00:00 2001 > From: Andrew Pinski > Date: Wed, 12 Feb 2020 11:42:57 + > Subject: [PATCH] MIPS: Fix __builtin_longjmp (PR 64242) > > The problem here is mips has its own builti

Re: [PATCH v2] [RISC-V] Add support for TLS stack protector canary access

2020-07-27 Thread Kito Cheng via Gcc-patches
Add testcase later is OK to me. On Tue, Jul 28, 2020 at 6:55 AM Jim Wilson wrote: > > On Sun, Jul 19, 2020 at 7:04 PM cooper wrote: > > Ping > > > > On 2020/7/13 下午4:15, cooper wrote: > > > gcc/ > > > * config/riscv/riscv-opts.h (stack_protector_guard): New enum. > > > * config/riscv

RE: [PATCH] RISC-V: Add ZFINX support

2020-07-27 Thread wangtao (CH)
Hi Jim, Thanks for your comments, We will try to solve the copyright problem and send the patch as soon as possible. Thanks, Tao Wang > -Original Message- > From: Jim Wilson [mailto:j...@sifive.com] > Sent: Tuesday, July 28, 2020 6:36 AM > To: wangtao (CH) > Cc: gcc-patches@gcc.gnu.or

[PATCH, rs6000] Add non-relative jump table support for 64bit rs6000

2020-07-27 Thread HAO CHEN GUI via Gcc-patches
Hi, This patch adds non-relative jump table support for 64bit rs6000. It implements ASM_OUTPUT_ADDR_VEC_ELT and corresponding expansion of jump table instruction for 64bit rs6000. We want to put non-relative jump table in data.rel.ro section for rs6000. So I add a new target hook - TARGET_ASM

Re: [PATCH] [RFC] vect: Fix infinite loop while determining peeling amount

2020-07-27 Thread Richard Biener via Gcc-patches
On Mon, Jul 27, 2020 at 4:20 PM Stefan Schulze Frielinghaus wrote: > > On Mon, Jul 27, 2020 at 12:29:11PM +0200, Richard Biener wrote: > > On Mon, Jul 27, 2020 at 11:45 AM Richard Sandiford > > wrote: > > > > > > Richard Biener writes: > > > > On Mon, Jul 27, 2020 at 11:09 AM Richard Sandiford >

<    1   2