Failed to build llvm 8.0.0 after upgrade gcc to 9.1.0

Hi Expert, I encounter a build error for llvm on ppc after upgrade gcc to 9.1.0 | collect2: fatal error: ld terminated with signal 11 [Segmentation fault] The linker used here is powerpc-poky-linux-g++. # powerpc-poky-linux-g++ -v Using built-in specs. COLLECT_GCC=powerpc-poky-linux-g++ COLLEC

[Fortran,wwwdocs] readings.html - updates to www.fortran.org

www.fortran.org made a number of changes to their site which now defaults to https, does not carry the FORTRAN 77 Standard any more (that I could find it), and removed some other material. This updates readings.html accordingly, and also streamlines the Fortran section there a bit to reduce those

Re: [C++ PATCH] PR c++/90449 - add -Winaccessible-base option.

On 6/10/19 12:02 PM, Matthew Beliveau wrote: if (!uniquely_derived_from_p (basetype, t)) - warning (OPT_Wextra, "virtual base %qT inaccessible in %qT due " - "to ambiguity", basetype, t); + warning (OPT_Winaccessible_base, "virtual base %qT inaccessible in

Re: [C++ PATCH] Avoid constexpr garbage for implicit conversion to void.

On Fri, Jun 7, 2019 at 5:08 PM Jason Merrill wrote: > > All expression statements and some other places express implicit conversion to > void with a CONVERT_EXPR. There's no reason to build up a new one as part of > constexpr evaluation. > > The ADDR_EXPR change also avoids a bit of garbage by di

[PING^4] [PATCH V3] PR88497 - Extend reassoc for vector bit_field_ref

Hi, Gentle ping again. Thanks! Kewen on 2019/5/21 上午10:02, Kewen.Lin wrote: > Hi, > > Gentle ping again. Thanks! > > > Kewen > > on 2019/5/5 下午2:15, Kewen.Lin wrote: >> Hi, >> >> I'd like to gentle ping for this patch: >> https://gcc.gnu.org/ml/gcc-patches/2019-03/msg00966.html >> >> OK fo

Re: [PATCH v3 2/3] Add predict_doloop_p target hook

>> If my understanding on this question is correct, IMHO we should try to make >> IVOPTs conservative than optimistic, since once the predict is wrong from >> too optimistic decision, the costing on the doloop use is wrong, it's very >> possible to affect the global optimal set. It looks we don't

[PATCH V8] Remove empty loop with assumed finiteness (PR tree-optimization/89713)

Reformat to comply with gcc coding style. Feng --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 37aab79..87cc125 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,16 @@ +2019-06-04 Feng Xue + + PR tree-optimization/89713 + * doc/invoke.texi (-ffinite-loops): Document ne

[PATCH V4] Find constant definition for by-ref argument using dominance information (PR ipa/90401)

> For future reference, there should be two spaces at the end of the sentence > before */.  You can use gcc/contrib/check_GNU_style.sh foo.patch to catch > stuff like this before posting a final patch. It's good. Thanks for pointing it out. Feng --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog ind

C++ PATCH for c++/90825 - endless recursion when evaluating sizeof

This test segvs since r269078, this hunk in particular: @@ -4581,8 +4713,9 @@ cxx_eval_constant_expression (const constexpr_ctx *ctx, tree t, break; case SIZEOF_EXPR: - r = fold_sizeof_expr (t); - VERIFY_CONSTANT (r); + r = cxx_eval_constant_expression (ctx, fold_sizeo

Re: [PATCH V3] Find constant definition for by-ref argument using dominance information (PR ipa/90401)

On Tue, Jun 11, 2019 at 01:30:41AM +, Feng Xue OS wrote: > Some minor changes. > > Feng > > --- > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 526ed45be89..469b54ddf93 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,14 @@ > +2019-06-04 Feng Xue > + > + PR ipa/9040

[PATCH V3] Find constant definition for by-ref argument using dominance information (PR ipa/90401)

Some minor changes. Feng --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 526ed45be89..469b54ddf93 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,14 @@ +2019-06-04 Feng Xue + + PR ipa/90401 + * ipa-prop.c (add_to_agg_contents_list): New function. + (clobber_by_

Re: [PATCH, v2] Disable PowerPC pc-relative support until the code is checked in

On Fri, Jun 07, 2019 at 07:31:39PM -0400, Michael Meissner wrote: > Ok, here is the revised version of the patch. -mcpu=future sets > -mprefixed-addr but not -mpcrel. Using either -mpcrel or -mprefixed-addr does > not set -mcpu=future. I tweaked the tests for -mpcrel and -mprefixed-addr > somewh

[PATCH] Fix i686-linux bootstrap (was Re: [PATCH 1/2] Add alloc_size for libiberty memory allocation functions.)

On Fri, Jun 07, 2019 at 09:07:39AM +0200, Martin Liška wrote: > 2019-06-07 Martin Liska > > * ansidecl.h: > (ATTRIBUTE_RESULT_SIZE_1): Define new macro. > (ATTRIBUTE_RESULT_SIZE_2): Likewise. > (ATTRIBUTE_RESULT_SIZE_1_2): Likewise. > * libiberty.h (xmalloc): Add R

Re: Test for C++20 p0858 - ConstexprIterator requirements.

On 6/10/19 2:43 AM, Ville Voutilainen wrote: On Mon, 10 Jun 2019 at 02:53, Ed Smith-Rowland <3dw...@verizon.net> wrote: Darn it, I had those constexpr lib patches in tree. Attached are what I just committed to gcc-9 and passes there. Those std::copy didn't really add anything anyway. They adde

Re: Fix build with inhibit_libc (was Re: [PATCH 2/4] Implement N disk counters for single value and indirect call counters.)

On Mon, Jun 10, 2019 at 08:02:26PM +0200, Jakub Jelinek wrote: > This change broke build without target libc. > > ../../../../libgcc/libgcov-merge.c:37:6: error: conflicting types for > ‘__gcov_merge_single’ >37 | void __gcov_merge_single (gcov_type *counters __attribute__ > ((unused))) >

[C++ PATCH] Fix C++ wrong-code due to constant_value_1 (PR c++/90810)

Hi! My PR85077 + if (VECTOR_TYPE_P (TREE_TYPE (x))) + x = fold (x); cp_fold change apparently broke the following and similar testcases, if split_nonconstant_init splits the initialization into a runtime part and some CONSTRUCTOR, it clears TREE_READONLY on it, and constant_value_1 h

[PATCH] Fix nvptx_output_softstack_switch (PR target/90811)

Hi! In PTX, and accepts only .pred, .b16, .b32 and .b64 types, not .u64 etc. Fixed thusly, bootstrapped/regtested on x86_64-linux with nvptx-none offloading, ok for trunk and 9.2/8.4? I'll try to figure out tomorrow if we can avoid the overaligned variables in this specific testcase, but generall

Go patch committed: Support inlining functions that use index expressions

This patch to the Go frontend supports inlining functions that use index expressions. It also moves the determine_types pass on an inlined function body to one place, rather than doing it ad hoc as needed. This adds 79 new inlinable functions in the standard library, such as bytes.HasPrefix and b

Go patch committed: Make heap expressions's write barrier conditional

This patch by Cherry Zhang avoids always invoking the write barrier for a heap expression. Heap_expression::do_get_backend emits an unconditional write barrier if the type has pointers and it is not a stack allocation. This patch changes it to use a write barrier for the assignment only when writ

Go patch committed: Permit inlining functions with labels and goto statements

This patch to the Go fronted permits inlining functions with labels and goto statements. This permits inlining functions with for loops and some switches, as they are lowered to if and goto statements before exporting them. This by itself only adds three new inlinable functions in the standard li

Re: [PATCH] include MEM_REF type in tree dumps (PR 90676)

On 6/10/19 1:29 PM, Jakub Jelinek wrote: On Mon, Jun 10, 2019 at 01:23:28PM -0600, Martin Sebor wrote: + else if (integer_zerop (TREE_OPERAND (node, 1)) + /* Dump the types of INTEGER_CSTs explicitly, for we can't + infer them and MEM_ATTR caching will share MEM_REFs +

Re: [C++ PATCH] Added test for c++/87250

On Mon, Jun 10, 2019 at 04:27:32PM -0400, Matthew Beliveau wrote: > My test is the result of reducing a file with just "include ", > just like in the PR. > > It was fixed in r269059 which added a similar test, except mine does > not use inheritance and is a bit simpler. So I figured it wouldn't >

Go patch committed: Use gcWriteBarrier for pointer-shaped struct/array

This patch by Cherry Zhang changes the Go frontend to use gcWriteBarrier for pointer-shaped struct/array values. If a struct/array is pointer-shaped (i.e. having a single field that is pointer-shaped), we can use gcWriteBarrier instead of typedmemmove for the write barrier. Bootstrapped and ran G

[C++ PATCH] Added test for c++/87250

My test is the result of reducing a file with just "include ", just like in the PR. It was fixed in r269059 which added a similar test, except mine does not use inheritance and is a bit simpler. So I figured it wouldn't hurt to cover more scenarios for this ICE. I ran the test in my build directo

Re: [PATCH] Improve PTA flow-sensitivity (for the return stmt)

On Wed, 5 Jun 2019, Richard Biener wrote: The following was inspired by Marins work on escapes of locals and the discussion there. It teaches points-to analysis that the point of function return is special and thus escapes through that a) do not influence other points-to solutions, b) can be pr

Go patch committed: Make escape analysis work with imported functions

This patch by Cherry Zhang fixes the Go frontend to make escape analysis work with imported inlineable functions. The escape analysis pass was written before we imported inlineable function bodies, and in some places it skipped functions that are not in the local package. Now that there are impor

Re: [PATCH] include MEM_REF type in tree dumps (PR 90676)

On Mon, Jun 10, 2019 at 01:23:28PM -0600, Martin Sebor wrote: > + else if (integer_zerop (TREE_OPERAND (node, 1)) > +/* Dump the types of INTEGER_CSTs explicitly, for we can't > + infer them and MEM_ATTR caching will share MEM_REFs > + with differently-typed op0s. */ >

Re: [PATCH] include MEM_REF type in tree dumps (PR 90676)

The attached update adds the MEM_REF type in pointy braces like in the -gimple format, but without the access size: MEM [(char *)&a] = 1; Martin On 6/4/19 4:57 AM, Richard Biener wrote: On Mon, Jun 3, 2019 at 5:13 PM Martin Sebor wrote: On 6/3/19 4:34 AM, Richard Biener wrote: On Mon, J

[PATCH, i386]: Improve and modernize ABS/NEG patterns

Attached patch improves ABS/NEG expander to emit clobber that enables ABS/NEG with integer registers also when SSE is disabled. The patch modernizes ABS/NEG insn patterns and splitters to use absneg code iterator and relevant mode iterators. Additionally, the patch enables three operand AVX instruc

Re: [PATCH][RFC] Sanitize equals and hash functions in hash-tables.

On Mon, Jun 10, 2019 at 3:08 AM Martin Liška wrote: > On 6/7/19 11:43 PM, Jason Merrill wrote: > > On Fri, Jun 7, 2019 at 8:14 AM Martin Liška wrote: > >> On 6/7/19 2:09 PM, Richard Biener wrote: > >>> On Fri, Jun 7, 2019 at 2:03 PM Martin Liška wrote: > On 6/7/19 10:57 AM, Richard Biener w

Fix build with inhibit_libc (was Re: [PATCH 2/4] Implement N disk counters for single value and indirect call counters.)

On Fri, Jun 07, 2019 at 11:10:59AM +0200, Martin Liška wrote: > libgcc/ChangeLog: > > 2019-06-04 Martin Liska > > * Makefile.in: Add __gcov_one_value_profiler_v2, > __gcov_one_value_profiler_v2_atomic and > __gcov_indirect_call_profiler_v4. > * libgcov-merge.c (__gcov_m

[patch][aarch64]: add intrinsics for vld1(q)_x4 and vst1(q)_x4

Greetings, This patch adds the intrinsic functions for: - vld1__x4 - vst1__x4 - vld1q__x4 - vst1q__x4 Bootstrapped and tested on aarch64-none-linux-gnu. Ok for trunk? If yes, I don't have any commit rights, so can someone please commit it on my behalf. Cheers, Syl gcc/ChangeLog: 2019-06-10

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

Hi Paul, I cannot see anything wrong with the optimized code and valgrind gives a clean bill of health on x86_64. We need help of somebody with access to an arm/aarch64 device. I'm currently running a bootstrap on an aarch64 machine. These are not known to be the fastest of machines, but it

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

Hi Christophe, I cannot see anything wrong with the optimized code and valgrind gives a clean bill of health on x86_64. We need help of somebody with access to an arm/aarch64 device. Cheers Paul On Mon, 10 Jun 2019 at 14:37, Christophe Lyon wrote: > > On Mon, 10 Jun 2019 at 13:05, Paul Richar

Re: [C++ PATCH] PR c++/90449 - add -Winaccessible-base option.

Hello, i changed the doc/invoke.texi like you suggested. Let me know If I missed anything! Thank you, Matthew Beliveau On Mon, Jun 10, 2019 at 11:39 AM Martin Sebor wrote: > > On 6/7/19 2:10 PM, Matthew Beliveau wrote: > > This patch adds a new warning option: Winaccessible-base, so that > > us

Re: [PATCH] Fix up g++.dg/cpp*/feat-cxx*.C tests (PR testsuite/90772, take 2, was: [PATCH] Avoid unnecessary inclusion of header)

On 06/06/19 22:34 +0200, Jakub Jelinek wrote: On Thu, Jun 06, 2019 at 10:19:52PM +0200, Jakub Jelinek wrote: Not only those, but apparently also ::size_t or std::plus; this broke FAIL: g++.dg/cpp1y/feat-cxx14.C (test for excess errors) FAIL: g++.dg/cpp1z/feat-cxx1z.C -std=gnu++17 (test for ex

Re: [C++ PATCH] PR c++/90449 - add -Winaccessible-base option.

On 6/7/19 2:10 PM, Matthew Beliveau wrote: This patch adds a new warning option: Winaccessible-base, so that users are able to selectively control the warning. The warning is enabled by default; however, for the virtual bases' warning, it only triggers with -Wextra flag. With few exceptions the

Re: [PATCH] PR other/90695 reduce testcase to remove library dependency

On 6/6/19 6:52 PM, Jonathan Wakely wrote: This reproduces the original ICE fixed by r178857 (tested at r178852 and r178860), without depending on a libstdc++ header that keeps changing. The number of errors differs between C++14 and C++17 modes, so the fixed test uses dg-excess-errors to match a

Re: [C++ PATCH] PR c++/90449 - add -Winaccessible-base option.

On Mon, Jun 10, 2019 at 10:19:51AM -0400, Matthew Beliveau wrote: > Hello, > > I've changed the name of warn_about_ambiguous_bases to > maybe_warn_about_inaccessible_bases. > > I've attached the new patch below. > > Best, > Matthew Beliveau > > On Fri, Jun 7, 2019 at 4:42 PM Paolo Carlini wrot

[committed, obvious] gcov-tool: Mark {merge,rewrite}_usage with noreturn attribute

This makes similar functions merge_usage, rewrite_usage, and overlap_usage all have ATTRIBUTE_NORETURN. (The latter was marked with it in r264562, the other two were not). Checked in as r272119. Index: gcc/ChangeLog === --- gcc/Chan

Re: [C++ PATCH] PR c++/90449 - add -Winaccessible-base option.

Hello, I've changed the name of warn_about_ambiguous_bases to maybe_warn_about_inaccessible_bases. I've attached the new patch below. Best, Matthew Beliveau On Fri, Jun 7, 2019 at 4:42 PM Paolo Carlini wrote: > > Hi, > > On 07/06/19 22:31, Marek Polacek wrote: > > On Fri, Jun 07, 2019 at 10:20

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

On Mon, 10 Jun 2019 at 13:05, Paul Richard Thomas wrote: > > Hi Christophe, > > I'll have a think about this tonight. Is valgrind or some similar > available for arm/aarch64? Yes, valgrind is available. I don't know if it's installed on the machines in the GCC computer farm though. Christophe

[PATCH V2] Find constant definition for by-ref argument using dominance information (PR ipa/90401)

--- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 526ed45be89..469b54ddf93 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,14 @@ +2019-06-04 Feng Xue + + PR ipa/90401 + * ipa-prop.c (add_to_agg_contents_list): New function. + (clobber_by_agg_contents_list_p): Lik

[PATCH] Disable htable sanitization in pt.c (PR c++/87847).

Hi. The patch is about disablement of hash_table sanitization in pt.c. It's a workaround and I hope Jason can look deeper at the issue. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/ChangeLog: 2019-06-10 Martin Liska

Handle structural equality of pointers in same_type_for_tbaa_p

Hi, this is kind of minimal patch to handle pointers in access paths. I do not include logic for arrays nor attempt to deal better with incomparable pointers (such as pointers to two different incomplete record types). A lazy way would be to simply compare alias sets, but I want to handle more bec

Re: C++ PATCH to implement deferred parsing of noexcept-specifiers (c++/86476, c++/52869)

Ping. On Mon, Jun 03, 2019 at 09:01:37PM -0400, Marek Polacek wrote: > On Tue, May 28, 2019 at 11:46:53AM -0400, Jason Merrill wrote: > > > @@ -20515,7 +20524,13 @@ cp_parser_init_declarator (cp_parser* parser, > > > /*asmspec=*/NULL_TREE, > > >

[committed] Add parsing and diagnostics for #pragma omp scan and reduction(inscan, ...)

Hi! The following patch does parsing and diagnostics for C/C++ #pragma omp scan and reduction(inscan, ...) clauses. omp-low.c and further handling is not implemented so far, we just sorry_at on those constructs if we don't diagnose any errors on those. Tested on x86_64-linux, committed to trunk.

[PATCH][OBVIOUS] Update __gcov_* scanned function name in a test-case.

Hi. The patch is fixing scanned name of a function in a test-case. The patch fixes the test-case. Martin gcc/testsuite/ChangeLog: 2019-06-10 Martin Liska * gcc.dg/no_profile_instrument_function-attr-1.c: Fix function name. --- gcc/testsuite/gcc.dg/no_profile_instrument_func

Allow conversions in X/[ex]4 < Y/[ex]4

Hello, if we compare for instance size() and capacity() for a std::vector, we get this kind of pattern. I started with just nop conversions and got a bit carried away handling all those cases. Bootstrap+regtest on x86_64-pc-linux-gnu. 2019-06-11 Marc Glisse gcc/ * match.pd (X/[ex

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

Hi Christophe, I'll have a think about this tonight. Is valgrind or some similar available for arm/aarch64? Many thanks for flagging it up. Paul On Mon, 10 Jun 2019 at 10:07, Christophe Lyon wrote: > > On Sat, 8 Jun 2019 at 18:25, Andrew Benson > wrote: > > > > Thanks Paul for the quick fix!

Re: [PATCH] Construct ipa_reduced_postorder always for overwritable (PR ipa/89009).

> Hi. > > IPA pure const should always construct ipa_reduced_postorder with > possibility to cross AVAIL_INTERPOSABLE boundary. The pass itself > can then properly stop propagation on these symbols. > > The patch is pre-approved by Honza. > Patch can bootstrap on x86_64-linux-gnu and survives reg

[C++ Patch] More grokdeclarator locations fixes

Hi, this one requires my last patch, uses id_loc in a few additional places in the last part of the function. Most of the changes should be straightforward, I only want to mention the "typedef name may not be a nested-name-specifier" change, where, not using input_location means that for a t

[PATCH][OBVIOUS] Add missing ATTR_UNUSED (PR bootstrap/90808).

Hi. This is addition of a missing ATTRIBUTE_UNUSED that is needed as the usage is conditional (#ifdef). I'm going to install the patch. Thanks, Martin libgcc/ChangeLog: 2019-06-10 Martin Liska PR bootstrap/90808 * libgcov.h: Add ATTRIBUTE_UNUSED. --- libgcc/libgcov.h | 2 +

Re: RFA: Synchronize top level files with binutils

Hi Richard, OK, here is a resubmission of my patch with just the addition of the libctf patches this time. (Sorry about the previous bad patch). Tested with a bootstrap and a normal build. OK to apply ? Cheers Nick 2019-06-07 Nick Clifton Import these changes from the binut

[PATCH][gcc] libgccjit: check result_type in gcc_jit_context_new_binary_op

Hi all, I would like to propose this patch to check for the return type of binary operators to be a numeric type. Not doing so can lead the compiler into funny crashes. Does not introduce regressions running make check-jit. OK for trunk? Bests Andrea 2019-06-09 Andrea Corallo andrea.cora...

Re: [Patch, fortran] PR90786 - [7/8/9/10 Regression] ICE on procedure pointer assignment to function with class pointer result

On Sat, 8 Jun 2019 at 18:25, Andrew Benson wrote: > > Thanks Paul for the quick fix! > > On Saturday, June 8, 2019 4:56:46 PM PDT Paul Richard Thomas wrote: > > Committed as obvious in revision 272084. > > > > The problem was that the lhs symbol itself was not being checked as a > > proc_pointer -

Re: [PATCH 1/2] [ARC] Improve code gen when compiling for size

Committed with your feedback taken into account. Thank you for your review, Claudiu On Sat, Jun 8, 2019 at 12:57 AM Jeff Law wrote: > > On 6/6/19 1:42 AM, Claudiu Zissulescu wrote: > > When optimizing for size, try to avoid using long immediate by > > employing alternative (short) instructions. >

Re: [PATCH 2/2] [ARC] Update RTX costs.

Thank you for your review! Patch committed, Claudiu On Sat, Jun 8, 2019 at 12:57 AM Jeff Law wrote: > > On 6/6/19 1:42 AM, Claudiu Zissulescu wrote: > > Update RTX costs to reflect better the ARC architecture. > > > > Ok to apply? > > Claudiu > > > > gcc/ > > -xx-xx Claudiu Zissulescu > >

Re: [PATCH 2/4] Implement N disk counters for single value and indirect call counters.

On 6/7/19 3:58 PM, Jan Hubicka wrote: >> >> Because I removed hist->hvalue.counters[2] where we stored total number of >> executions. >> It's back again so that _atomic suffix version makes sense again. > > OK and we do not care about race conditions on the other two counters? We do care, but on

[PATCH, testsuite, obvious] Require alias support in a test using it.

attr-copy-6.c fails on Darwin which doesn’t have alias support. Fixed by adding the dg-requires line. tested on x86_64-darwin16, x86_64-linux-gnu applied as obvious to mainline. thanks Iain 2019-06-10 Iain Sandoe * gcc.dg/attr-copy-6.c: Require alias support. diff --git a/gcc/testsui

Re: [doc] extend.texi -- clarify no_sanitize

On 6/9/19 5:42 PM, Gerald Pfeifer wrote: > In some interpretations "not...of all" may not be seen as > 100% clear, so I propose to use "not...of any". > > On the way fix the grammar in the next sentence. > > Sandra, and Martin as the original author, does this makes > sense for you? I'm fine w

Re: [PATCH][RFC] Sanitize equals and hash functions in hash-tables.

On 6/7/19 11:43 PM, Jason Merrill wrote: > On Fri, Jun 7, 2019 at 8:14 AM Martin Liška wrote: >> >> On 6/7/19 2:09 PM, Richard Biener wrote: >>> On Fri, Jun 7, 2019 at 2:03 PM Martin Liška wrote: On 6/7/19 10:57 AM, Richard Biener wrote: > On Mon, Jun 3, 2019 at 3:35 PM Martin Liška