gcc.git
5 years agoSkip libstdc++ debug build in early bootstrap stages
Jonathan Wakely [Thu, 20 Jun 2019 14:17:57 +0000 (15:17 +0100)]
Skip libstdc++ debug build in early bootstrap stages

As mentioned in PR 90770, this is a patch that Debian have been carrying
for some time. The additional unoptimized copies of libstdc++ libs that
get built during each stage are never going to be used, so don't bother
building them.

For a profiled bootstrap this means we won't train the compiler on the
unoptimized library code with assertions enabled, but that doesn't seem
like a big problem, as the same code has already been compiled once for
the main libstdc++ library.

* acinclude.m4 (GLIBCXX_ENABLE_DEBUG): Only do debug build for final
stage of bootstrap.
* configure: Regenerate.

From-SVN: r272509

5 years agoQualify calls to __never_valueless in <variant>
Jonathan Wakely [Thu, 20 Jun 2019 14:17:51 +0000 (15:17 +0100)]
Qualify calls to __never_valueless in <variant>

* include/std/variant (_Variant_storage, _Extra_visit_slot_needed):
Qualify calls to __never_valueless.

From-SVN: r272508

5 years agore PR c++/89873 (internal compiler error: unexpected expression of kind implicit_conv...
Marek Polacek [Thu, 20 Jun 2019 12:22:25 +0000 (12:22 +0000)]
re PR c++/89873 (internal compiler error: unexpected expression of kind implicit_conv_expr)

PR c++/89873
* g++.dg/cpp1y/noexcept1.C: New test.

From-SVN: r272507

5 years agore PR fortran/90937 (ICE: in gfc_get_symbol_decl, at fortran/trans-decl.c:1538)
Thomas Koenig [Thu, 20 Jun 2019 11:56:50 +0000 (11:56 +0000)]
re PR fortran/90937 (ICE: in gfc_get_symbol_decl, at fortran/trans-decl.c:1538)

2019-06-20  Thomas Koenig  <tkoenig@gcc.gnu.org>

PR fortran/90937
* trans-types.c (get_formal_from_actual_arglist): Get symbol from
current namespace so it will be freed later.  If symbol is of type
character, get an empty character length.

2019-06-20  Thomas Koenig  <tkoenig@gcc.gnu.org>

PR fortran/90937
* gfortran.dg/external_procedure_4.f90: New test.

From-SVN: r272506

5 years ago[testsuite] Add missing dg-require-effective-target global_constructor
Tom de Vries [Thu, 20 Jun 2019 10:37:18 +0000 (10:37 +0000)]
[testsuite] Add missing dg-require-effective-target global_constructor

Add missing dg-require-effective-target global_constructor.

Tested on nvptx.

2019-06-20  Tom de Vries  <tdevries@suse.de>

* gcc.dg/pr90866-2.c: Require global_constructor.

From-SVN: r272505

5 years ago[testsuite] Add missing dg-require-effective-target nonlocal_goto
Tom de Vries [Thu, 20 Jun 2019 10:37:05 +0000 (10:37 +0000)]
[testsuite] Add missing dg-require-effective-target nonlocal_goto

Add missing dg-require-effective-target nonlocal_goto.

Tested on nvptx.

2019-06-20  Tom de Vries  <tdevries@suse.de>

* gcc.c-torture/compile/pr89280.c: Require nonlocal_goto.
* gcc.dg/pr88870.c: Same.
* gcc.dg/pr90082.c: Same.

From-SVN: r272504

5 years ago[testsuite] Add missing dg-require-effective-target indirect_jumps
Tom de Vries [Thu, 20 Jun 2019 10:36:49 +0000 (10:36 +0000)]
[testsuite] Add missing dg-require-effective-target indirect_jumps

Add missing dg-require-effective-target indirect_jumps.

Tested on nvptx.

2019-06-20  Tom de Vries  <tdevries@suse.de>

* gcc.dg/pr89737.c: Require indirect_jumps.
* gcc.dg/torture/pr87693.c: Same.
* gcc.dg/torture/pr89135.c: Same.
* gcc.dg/torture/pr90071.c: Same.

From-SVN: r272503

5 years ago[testsuite] Add missing dg-require-effective-target label_values
Tom de Vries [Thu, 20 Jun 2019 10:36:35 +0000 (10:36 +0000)]
[testsuite] Add missing dg-require-effective-target label_values

Add missing dg-require-effective-target label_values.

Tested on nvptx.

2019-06-20  Tom de Vries  <tdevries@suse.de>

* gcc.c-torture/compile/pr89280.c: Require label_values.
* gcc.dg/pr89737.c: Same.
* gcc.dg/pr90082.c: Same.
* gcc.dg/torture/pr89135.c: Same.
* gcc.dg/torture/pr89247.c: Same.
* gcc.dg/torture/pr90071.c: Same.

From-SVN: r272502

5 years ago[testsuite] Add missing dg-require-effective-target alloca
Tom de Vries [Thu, 20 Jun 2019 10:36:18 +0000 (10:36 +0000)]
[testsuite] Add missing dg-require-effective-target alloca

Add missing dg-require-effective-target alloca.

Tested on nvptx.

2019-06-20  Tom de Vries  <tdevries@suse.de>

* gcc.c-torture/compile/pr77754-1.c: Require alloca.
* gcc.c-torture/compile/pr77754-2.c: Same.
* gcc.c-torture/compile/pr77754-3.c: Same.
* gcc.c-torture/compile/pr77754-4.c: Same.
* gcc.c-torture/compile/pr77754-5.c: Same.
* gcc.c-torture/compile/pr77754-6.c: Same.
* gcc.c-torture/compile/pr87110.c: Same.
* gcc.c-torture/execute/pr86528.c: Same.
* gcc.dg/Walloca-larger-than-2.c: Same.
* gcc.dg/Walloca-larger-than.c: Same.
* gcc.dg/Warray-bounds-41.c: Same.
* gcc.dg/Wrestrict-17.c: Same.
* gcc.dg/Wstrict-overflow-27.c: Same.
* gcc.dg/Wstringop-truncation-3.c: Same.
* gcc.dg/pr78902.c: Same.
* gcc.dg/pr87099.c: Same.
* gcc.dg/pr87320.c: Same.
* gcc.dg/pr89045.c: Same.
* gcc.dg/strlenopt-62.c: Same.
* gcc.dg/tree-ssa/alias-37.c: Same.

From-SVN: r272501

5 years agoFix outdated reference to C++17 draft in the docs
Jonathan Wakely [Thu, 20 Jun 2019 09:13:03 +0000 (10:13 +0100)]
Fix outdated reference to C++17 draft in the docs

* doc/xml/manual/status_cxx2017.xml: Fix outdated reference to
C++17 working draft.

From-SVN: r272500

5 years agoImprove tests for std::vector<bool> printer
Jonathan Wakely [Thu, 20 Jun 2019 09:04:55 +0000 (10:04 +0100)]
Improve tests for std::vector<bool> printer

The current tests wouldn't notice if the vector<bool> contents were
printed in reverse, because it would read the same forwards and
backwards. Change the content so the tests would fail if that happened.

* testsuite/libstdc++-prettyprinters/simple.cc: Use non-palindromic
vector<bool> for test.
* testsuite/libstdc++-prettyprinters/simple11.cc: Likewise.

From-SVN: r272499

5 years agoDecrease hash-table-verification-limit from 100 to 10.
Martin Liska [Thu, 20 Jun 2019 07:41:42 +0000 (09:41 +0200)]
Decrease hash-table-verification-limit from 100 to 10.

2019-06-20  Martin Liska  <mliska@suse.cz>

* params.def (PARAM_HASH_TABLE_VERIFICATION_LIMIT): Decrease
to 10.

From-SVN: r272498

5 years agotree-vect-stmts.c (enum scan_store_kind): New type.
Jakub Jelinek [Thu, 20 Jun 2019 07:06:08 +0000 (09:06 +0200)]
tree-vect-stmts.c (enum scan_store_kind): New type.

* tree-vect-stmts.c (enum scan_store_kind): New type.
(scan_store_can_perm_p): Change last argument from int * to
vec<enum scan_store_kind> *, record precisely which permutations
need whole vector left shift or that plus VEC_COND_EXPR.
(vectorizable_scan_store): Adjust caller, use whole vector left shift
and additional VEC_COND_EXPR only for those iterations that need it.

From-SVN: r272497

5 years agofix ARM --with-fpu option checking and error message
Alexandre Oliva [Thu, 20 Jun 2019 05:55:34 +0000 (05:55 +0000)]
fix ARM --with-fpu option checking and error message

Fix the test for failure in parsecpu's checking of the --with-fpu
argument, and the error message that gets printed when the check
fails.

for  gcc/ChangeLog

* config.gcc: Fix ARM --with-fpu checking and error message.

From-SVN: r272496

5 years agoDaily bump.
GCC Administrator [Thu, 20 Jun 2019 00:16:14 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r272495

5 years agoFix non-standard behaviour of std::istream_iterator
Jonathan Wakely [Wed, 19 Jun 2019 22:57:10 +0000 (23:57 +0100)]
Fix non-standard behaviour of std::istream_iterator

The current implementation of istream_iterator allows the iterator to be
reused after reaching end-of-stream, so that subsequent reads from the
stream can succeed (e.g. if the stream state has been cleared and stream
position changed from EOF). The P0738R2 paper clarified that the
expected behaviour is to set the stream pointer to null after reaching
end-of-stream, preventing further reads.

This implements that requirement, and adds the new default constructor
to std::ostream_iterator.

* include/bits/stream_iterator.h (istream_iterator::_M_equal()): Make
private.
(istream_iterator::_M_read()): Do not check stream state before
attempting extraction. Set stream pointer to null when extraction
fails (P0738R2).
(operator==(const istream_iterator&, const istream_iterator&)): Change
to be a hidden friend of istream_iterator.
(operator!=(const istream_iterator&, const istream_iterator&)):
Likewise.
(ostream_iterator::ostream_iterator()): Add default constructor.
(ostream_iterator::ostream_iterator(ostream_type*, const C*)): Use
addressof.
* testsuite/24_iterators/istream_iterator/1.cc: New test.
* testsuite/24_iterators/ostream_iterator/1.cc: New test.
* testsuite/24_iterators/ostream_iterator/70766.cc: Also check
constructor taking a string.
* testsuite/24_iterators/ostream_iterator/requirements/constexpr.cc:
New test.

From-SVN: r272491

5 years agoHave std::vector printer's iterator return bool for vector<bool>
Michael Weghorn [Wed, 19 Jun 2019 22:57:06 +0000 (22:57 +0000)]
Have std::vector printer's iterator return bool for vector<bool>

Have the pretty-printer for 'std::vector<bool>' return a
value of type 'bool' rather than an 'int'.

This way, the type is clear and that can be used for better
display and a 'gdb.Value' constructed from the returned value
will have type 'bool' again, not e.g. 'long long' as happened
previously (at least with GDB 8.2.1 on amd64).

2019-06-19  Michael Weghorn  <m.weghorn@posteo.de>
    Jonathan Wakely  <jwakely@redhat.com>

PR libstdc++/90945
* python/libstdcxx/v6/printers.py (StdVectorPrinter._iterator): Use
values of type bool for vector<bool> elements.
* testsuite/libstdc++-prettyprinters/simple.cc: Test vector<bool>.
* testsuite/libstdc++-prettyprinters/simple11.cc: Likewise.

Co-Authored-By: Jonathan Wakely <jwakely@redhat.com>
From-SVN: r272490

5 years agoPR libstdc++/90920 restore previous checks for empty ranges
Jonathan Wakely [Wed, 19 Jun 2019 22:57:02 +0000 (23:57 +0100)]
PR libstdc++/90920 restore previous checks for empty ranges

The change in r263433 broke the contract of the __rotate functions, by no
longer accepting empty ranges. That means that callers which inlined the
old version of std::rotate (without checks) that end up linking to a new
definition of std::__rotate (also without checks) could perform a divide
by zero and crash.

This restores the old contract of the __rotate overloads.

PR libstdc++/90920 partially revert r263433
* include/bits/stl_algo.h (__rotate): Restore checks for empty ranges.
(rotate): Remove checks.
* testsuite/25_algorithms/rotate/90920.cc: New test.

From-SVN: r272489

5 years agore PR tree-optimization/90626 (fold strcmp(a, b) == 0 to zero when one string length...
Martin Sebor [Wed, 19 Jun 2019 21:46:09 +0000 (21:46 +0000)]
re PR tree-optimization/90626 (fold strcmp(a, b) == 0 to zero when one string length is exact and the other is unequal)

PR tree-optimization/90626

gcc/ChangeLog:
* tree-ssa-strlen.c (strxcmp_unequal): Fix typos.

From-SVN: r272487

5 years agoPR c++/60364 - noreturn after first decl not diagnosed.
Marek Polacek [Wed, 19 Jun 2019 21:27:45 +0000 (21:27 +0000)]
PR c++/60364 - noreturn after first decl not diagnosed.

* attribs.c (get_attribute_namespace): No longer static.
(decl_attributes): Avoid shadowing.  Preserve the C++11 form for C++11
attributes.
(attr_noreturn_exclusions): Make it extern.
* attribs.h (get_attribute_namespace): Declare.
* tree-inline.c (function_attribute_inlinable_p): Use
get_attribute_name.

* c-attribs.c (handle_noreturn_attribute): No longer static.
* c-common.h (handle_noreturn_attribute, attr_noreturn_exclusions):
Declare.
* c-format.c (check_function_format): Use get_attribute_name.

* decl.c (duplicate_decls): Give an error when a function is
declared [[noreturn]] after its first declaration.
* parser.c (cp_parser_std_attribute): Don't treat C++11 noreturn
attribute as equivalent to GNU's.
* tree.c (std_attribute_table): Add noreturn.

* g++.dg/warn/noreturn-8.C: New test.
* g++.dg/warn/noreturn-9.C: New test.
* g++.dg/warn/noreturn-10.C: New test.
* g++.dg/warn/noreturn-11.C: New test.

From-SVN: r272486

5 years agoPR tree-optimization/90626 - fold strcmp(a, b) == 0 to zero when one string length...
Martin Sebor [Wed, 19 Jun 2019 20:37:41 +0000 (20:37 +0000)]
PR tree-optimization/90626 - fold strcmp(a, b) == 0 to zero when one string length is exact and the other is unequal

gcc/ChangeLog:

PR tree-optimization/90626
* tree-ssa-strlen.c (strxcmp_unequal): New function.
(handle_builtin_string_cmp): Call it.

gcc/testsuite/ChangeLog:

PR tree-optimization/90626
* gcc.dg/strlenopt-65.c: New test.
* gcc.dg/strlenopt-66.c: New test.
* gcc.dg/strlenopt.h (strcmp, strncmp): Declare.

From-SVN: r272485

5 years ago[Darwin, specs] Tidy some more linker options.
Iain Sandoe [Wed, 19 Jun 2019 19:16:17 +0000 (19:16 +0000)]
[Darwin, specs] Tidy some more linker options.

pie, no-pie and rdynamic are driver options, we can process them in the
relevant place and drop them once dealt with.  There's no need to generate
a new header to process the "no_compact_unwind" which is applied on the
basis of the target system.

Support for the -pie, -no_pie and -no_compact_unwind options should ideally
be checked at configure time, however the status quo is to assert that linkers
capable of targeting the relevant systems support these options (i.e. we trust
that the user doesn't attempt to configure inappropriately).

TODO: check the availability of the linker opts in configure rather than
trusting to the user.

This will fix the fail of pie-7.c, which is a result of failing to handle the
no-pie driver option.

2019-06-19  Iain Sandoe  <iain@sandoe.co.uk>

* config/darwin.h (DRIVER_SELF_SPECS): Add RDYNAMIC, DARWIN_PIE_SPEC
and DARWIN_NOPIE_SPEC.
(RDYNAMIC): New, modified from DARWIN_EXPORT_DYNAMIC.
(DARWIN_PIE_SPEC): Collate from darwin.h and darwin9.h.
(DARWIN_NOPIE_SPEC): Collate from darwin10.h.
(DARWIN_NOCOMPACT_UNWIND): New from darwin10.h
(DARWIN_EXPORT_DYNAMIC): Delete.
* config/darwin10.h (LINK_GCC_C_SEQUENCE_SPEC): Move no_compact_unwind
and pie options processing to  darwin.h.
* config/darwin9.h (DARWIN_PIE_SPEC): Move pie processing to darwin.h

From-SVN: r272484

5 years agoPR translation/90156 - add linter check suggesting to replace %<%s%> with %qs
Martin Sebor [Wed, 19 Jun 2019 19:08:24 +0000 (19:08 +0000)]
PR translation/90156 - add linter check suggesting to replace %<%s%> with %qs

gcc/c-family/ChangeLog:

PR translation/90156
* c-format.c (function_format_info::format_type): Adjust type.
(function_format_info::is_raw): New member.
(decode_format_type): Adjust signature.  Handle "raw" diag attributes.
(decode_format_attr): Adjust call to decode_format_type.
Avoid a redundant call to convert_format_name_to_system_name.
Avoid abbreviating the word "arguments" in a diagnostic.
(format_warning_substr): New function.
(avoid_dollar_number): Quote dollar sign in a diagnostic.
(finish_dollar_format_checking): Same.
(check_format_info): Same.
(struct baltoks_t): New.
(c_opers, c_keywords, cxx_keywords, badwords, contrs): New arrays.
(maybe_diag_unbalanced_tokens, check_tokens, check_plain): New
functions.
(check_format_info_main): Call check_plain.  Use baltoks_t.  Call
maybe_diag_unbalanced_tokens.
(handle_format_attribute): Spell out the word "arguments" in
a diagnostic.

gcc/testsuite/ChangeLog:

PR translation/90156
* gcc.dg/format/gcc_diag-11.c: Enable.

From-SVN: r272483

5 years agore PR fortran/69499 ([F03] ICE-on-invalid on combining select type with wrong statement)
Steven G. Kargl [Wed, 19 Jun 2019 18:18:40 +0000 (18:18 +0000)]
re PR fortran/69499 ([F03] ICE-on-invalid on combining select type with wrong statement)

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/69499
* match.c (gfc_match_select_type):  SELECT TYPE is an executable
statement, and cannot appear in MODULE or SUBMODULE scope.

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/69499
* gfortran.dg/pr69499.f90: New test.
* gfortran.dg/module_error_1.f90: Update dg-error string.

From-SVN: r272482

5 years agore PR fortran/69398 ([OOP] ICE on class with duplicate dimension attribute specified)
Steven G. Kargl [Wed, 19 Jun 2019 18:04:46 +0000 (18:04 +0000)]
re PR fortran/69398 ([OOP] ICE on class with duplicate dimension attribute specified)

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/69398
* decl.c (attr_decl): Check for duplicate DIMENSION attribute for a
CLASS entity.

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/69398
* gfortran.dg/pr69398.f90: New test.

From-SVN: r272481

5 years agore PR fortran/87907 (ICE in resolve_contained_fntype, at fortran/resolve.c:587)
Steven G. Kargl [Wed, 19 Jun 2019 17:58:54 +0000 (17:58 +0000)]
re PR fortran/87907 (ICE in resolve_contained_fntype, at fortran/resolve.c:587)

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/87907
* resolve.c (resolve_contained_fntype): Do not dereference a NULL
pointer.

2019-06-19  Steven G. Kargl  <kargl@gcc.gnu.org>

PR fortran/87907
* gfortran.dg/pr87907.f90: New testcase.

From-SVN: r272480

5 years ago[Darwin] Fix two off-by-one errors in the driver.
Iain Sandoe [Wed, 19 Jun 2019 16:07:58 +0000 (16:07 +0000)]
[Darwin] Fix two off-by-one errors in the driver.

2019-06-19  Iain Sandoe  <iain@sandoe.co.uk>

        * config/darwin-driver.c (darwin_driver_init): Fix off-by-one errors
        in computing the number of options to be moved.

From-SVN: r272479

5 years agonetbsd-elf.h (SYSARCH_ARM_SYNC_ICACHE): New definition.
Maya Rashish [Wed, 19 Jun 2019 16:01:24 +0000 (16:01 +0000)]
netbsd-elf.h (SYSARCH_ARM_SYNC_ICACHE): New definition.

*  config/arm/netbsd-elf.h (SYSARCH_ARM_SYNC_ICACHE): New definition.
(CLEAR_INSN_CACHE) Use it.

From-SVN: r272478

5 years agoFix value category bugs in std::reduce
Jonathan Wakely [Wed, 19 Jun 2019 15:29:49 +0000 (16:29 +0100)]
Fix value category bugs in std::reduce

* include/std/numeric (reduce(Iter, Iter, T, BinOp)): Fix value
category used in invocable check.
(reduce(Iter, Iter, T)): Pass initial value as rvalue.
* testsuite/26_numerics/reduce/2.cc: New test.

From-SVN: r272477

5 years agocompiler: optimize string concatenations
Ian Lance Taylor [Wed, 19 Jun 2019 15:13:53 +0000 (15:13 +0000)]
compiler: optimize string concatenations

    runtime.concatstring{2,3,4,5} are just wrappers of concatstrings.
    These wrappers don't provide any benefit, at least in the C
    calling convention we use, where passing arrays by value isn't an
    efficient thing. Change it to always use concatstrings.

    Also, the cap field of the slice passed to concatstrings is not
    necessary. So change it to pass a pointer and a length directly,
    which is more efficient than passing a slice header by value.

    Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/182539

From-SVN: r272476

5 years agoi386.md (cmpstrnsi): Remove dead code.
Uros Bizjak [Wed, 19 Jun 2019 14:38:58 +0000 (16:38 +0200)]
i386.md (cmpstrnsi): Remove dead code.

* config/i386/i386.md (cmpstrnsi): Remove dead code.

From-SVN: r272474

5 years agoSimplify setjmp and non-local goto implementation (PR84521)
Wilco Dijkstra [Wed, 19 Jun 2019 12:52:43 +0000 (12:52 +0000)]
Simplify setjmp and non-local goto implementation (PR84521)

This fixes and simplifies the setjmp and non-local goto implementation.
Currently the virtual frame pointer is saved when using __builtin_setjmp or
a non-local goto.  Depending on whether a frame pointer is used, this may
either save SP or FP with an immediate offset.  However the goto or longjmp
always updates the hard frame pointer.

A receiver veneer in the original function then assigns the hard frame pointer
to the virtual frame pointer, which should, if it works correctly, again assign
SP or FP.  However the special elimination code in eliminate_regs_in_insn
doesn't do this correctly unless the frame pointer is used, and even if it
worked by writing SP, the frame pointer would still be corrupted.

A much simpler implementation is to always save and restore the hard frame
pointer.  This avoids 2 redundant instructions which add/subtract the virtual
frame offset.  A large amount of code can be removed as a result, including all
implementations of TARGET_BUILTIN_SETJMP_FRAME_VALUE (all of which already use
the hard frame pointer).  The expansion of nonlocal_goto on PA can be simplied
to just restore the hard frame pointer.

This fixes the most obvious issues, however there are still issues on targets
which define HARD_FRAME_POINTER_IS_FRAME_POINTER (arm, mips).
Each function could have a different hard frame pointer, so a non-local goto
may restore the wrong frame pointer (TARGET_BUILTIN_SETJMP_FRAME_VALUE could
be useful for this).

The i386 TARGET_BUILTIN_SETJMP_FRAME_VALUE was incorrect: if stack_realign_fp
is true, it would save the hard frame pointer value but restore the virtual
frame pointer which according to ix86_initial_elimination_offset can have a
non-zero offset from the hard frame pointer.

The ia64 implementation of nonlocal_goto seems incorrect since the helper
function moves the the frame pointer value into the static chain register
(so this patch does nothing to make it better or worse).

AArch64 + x86-64 bootstrap OK, new test passes on AArch64, x86-64 and Arm.

gcc/
PR middle-end/84521
* builtins.c (expand_builtin_setjmp_setup): Save
hard_frame_pointer_rtx.
(expand_builtin_setjmp_receiver): Do not emit sfp = fp move since we
restore fp.
* function.c (expand_function_start): Save hard_frame_pointer_rtx for
non-local goto.
* lra-eliminations.c (eliminate_regs_in_insn): Remove sfp = fp
elimination code.
(remove_reg_equal_offset_note): Remove unused function.
* reload1.c (eliminate_regs_in_insn): Remove sfp = hfp elimination
code.
* config/arc/arc.c (TARGET_BUILTIN_SETJMP_FRAME_VALUE): Remove.
(arc_builtin_setjmp_frame_value): Remove function.
* config/avr/avr.c  (TARGET_BUILTIN_SETJMP_FRAME_VALUE): Remove.
(avr_builtin_setjmp_frame_value): Remove function.
* config/i386/i386.c (TARGET_BUILTIN_SETJMP_FRAME_VALUE): Remove.
(ix86_builtin_setjmp_frame_value): Remove function.
* config/pa/pa.md (nonlocal_goto): Remove FP adjustment.
* config/sparc/sparc.c (TARGET_BUILTIN_SETJMP_FRAME_VALUE): Remove.
(sparc_builtin_setjmp_frame_value): Remove function.
* config/vax/vax.c (TARGET_BUILTIN_SETJMP_FRAME_VALUE): Remove.
(vax_builtin_setjmp_frame_value): Remove function.
* config/xtensa/xtensa.c (xtensa_frame_pointer_required): Force frame
pointer if has_nonlocal_label.

testsuite/
PR middle-end/84521
* gcc.c-torture/execute/pr84521.c: New test.

From-SVN: r272473

5 years agomd.texi: Document vec_shl_<mode> pattern.
Jakub Jelinek [Wed, 19 Jun 2019 10:00:04 +0000 (12:00 +0200)]
md.texi: Document vec_shl_<mode> pattern.

* doc/md.texi: Document vec_shl_<mode> pattern.
* optabs.def (vec_shl_optab): New optab.
* optabs.c (shift_amt_for_vec_perm_mask): Add shift_optab
argument, if == vec_shl_optab, check for left whole vector shift
pattern rather than right shift.
(expand_vec_perm_const): Add vec_shl_optab support.
* optabs-query.c (can_vec_perm_var_p): Mention also vec_shl optab
in the comment.
* tree-vect-generic.c (lower_vec_perm): Support permutations which
can be handled by vec_shl_optab.
* tree-vect-stmts.c (scan_store_can_perm_p): New function.
(check_scan_store): Use it.
(vectorizable_scan_store): If target can't do normal permutations,
try to use whole vector left shifts and if needed a VEC_COND_EXPR
after it.
* config/i386/sse.md (vec_shl_<mode>): New expander.

* gcc.dg/vect/vect-simd-8.c: If main is defined, don't include
tree-vect.h nor call check_vect.
* gcc.dg/vect/vect-simd-9.c: Likewise.
* gcc.dg/vect/vect-simd-10.c: New test.
* gcc.target/i386/sse2-vect-simd-8.c: New test.
* gcc.target/i386/sse2-vect-simd-9.c: New test.
* gcc.target/i386/sse2-vect-simd-10.c: New test.
* gcc.target/i386/avx2-vect-simd-8.c: New test.
* gcc.target/i386/avx2-vect-simd-9.c: New test.
* gcc.target/i386/avx2-vect-simd-10.c: New test.
* gcc.target/i386/avx512f-vect-simd-8.c: New test.
* gcc.target/i386/avx512f-vect-simd-9.c: New test.
* gcc.target/i386/avx512f-vect-simd-10.c: New test.

From-SVN: r272472

5 years agoomp-low.c (lower_rec_input_clauses): Handle references properly in inscan clauses.
Jakub Jelinek [Wed, 19 Jun 2019 08:32:09 +0000 (10:32 +0200)]
omp-low.c (lower_rec_input_clauses): Handle references properly in inscan clauses.

* omp-low.c (lower_rec_input_clauses): Handle references properly
in inscan clauses.
(lower_omp_scan): Likewise.
cp/
* cp-gimplify.c (cp_genericize_r): Handle OMP_CLAUSE_{IN,EX}CLUSIVE
like OMP_CLAUSE_SHARED.
testsuite/
* g++.dg/vect/simd-3.cc: New test.
* g++.dg/vect/simd-4.cc: New test.
* g++.dg/vect/simd-5.cc: New test.

From-SVN: r272471

5 years ago* g++.dg/ubsan/pr63956.C: Adjust expected diagnostics.
Jakub Jelinek [Wed, 19 Jun 2019 08:22:29 +0000 (10:22 +0200)]
* g++.dg/ubsan/pr63956.C: Adjust expected diagnostics.

From-SVN: r272470

5 years agoAdd new micro-benchmark for string operations.
Martin Liska [Wed, 19 Jun 2019 07:24:02 +0000 (09:24 +0200)]
Add new micro-benchmark for string operations.

2019-06-19  Martin Liska  <mliska@suse.cz>

* bench-stringop: New file.

From-SVN: r272469

5 years agocompiler: stack allocate a buffer for non-escaping string ops
Ian Lance Taylor [Wed, 19 Jun 2019 04:53:51 +0000 (04:53 +0000)]
compiler: stack allocate a buffer for non-escaping string ops

    For string concatenation, string to/from byte or rune slice
    conversion, and int to string conversion, if the result does not
    escape, we can allocate a small (32-element, or 4-byte for int to
    string) buffer on stack, and pass it to the runtime function. If
    the result fits in the buffer, it doesn't need to do a heap
    allocation.

    Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/182538

From-SVN: r272468

5 years agore PR fortran/89103 (Allow blank format items in format strings)
Jim MacArthur [Wed, 19 Jun 2019 03:02:21 +0000 (03:02 +0000)]
re PR fortran/89103 (Allow blank format items in format strings)

2019-06-19  Jim MacArthur  <jim.macarthur@codethink.co.uk>
    Mark Eggleston  <mark.eggleston@codethink.com>

PR fortran/89103
* gfortran.texi: Add -fdec-blank-format-item
* invoke.texi: Add option to list of options.
* invoke.texi: Add to section on Commas in FORMAT specifications.
* io.c (check_format): At FMT_RPAREN goto finished if
-fdec-blank-format-item otherwise set error string.
* lang.opt: Add new option.
* options.c (set_dec_flags): Add SET_BITFLAG for
flag_dec_format_defaults.

* gfortran.dg/dec_format_empty_item_1.f: New test.
* gfortran.dg/dec_format_empty_item_2.f: New test.
* gfortran.dg/dec_format_empty_item_3.f: New test.

Co-Authored-By: Mark Eggleston <mark.eggleston@codethink.com>
From-SVN: r272467

5 years agopr88834.c: Move from here...
Kugan Vivekanandarajah [Wed, 19 Jun 2019 01:49:24 +0000 (01:49 +0000)]
pr88834.c: Move from here...

gcc/testsuite/ChangeLog:

2019-06-19  Kugan Vivekanandarajah  <kugan.vivekanandarajah@linaro.org>

* gcc.target/aarch64/pr88834.c: Move from here...
* gcc.target/aarch64/sve/pr88834.c: ...to here.

From-SVN: r272466

5 years agotree-ssa-address.c (preferred_mem_scale_factor): Handle when mem_mode is BLKmode.
Kugan Vivekanandarajah [Wed, 19 Jun 2019 01:42:21 +0000 (01:42 +0000)]
tree-ssa-address.c (preferred_mem_scale_factor): Handle when mem_mode is BLKmode.

gcc/ChangeLog:

2019-06-19  Kugan Vivekanandarajah  <kugan.vivekanandarajah@linaro.org>

* tree-ssa-address.c (preferred_mem_scale_factor): Handle when
mem_mode is BLKmode.

From-SVN: r272465

5 years agoDaily bump.
GCC Administrator [Wed, 19 Jun 2019 00:16:39 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r272464

5 years agocompiler: avoid copy for string([]byte) conversion used in string concatenation
Cherry Zhang [Tue, 18 Jun 2019 23:55:50 +0000 (23:55 +0000)]
compiler: avoid copy for string([]byte) conversion used in string concatenation

    If a string([]byte) conversion is used immediately in a string
    concatenation, we don't need to copy the backing store of the
    byte slice, as the runtime function doesn't hold any reference
    to it.

    Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/182437

* go.dg/concatstring.go: New test.

From-SVN: r272460

5 years agoImplement new serial algorithms from Parallelism TS (P0024R2)
Jonathan Wakely [Tue, 18 Jun 2019 23:01:16 +0000 (00:01 +0100)]
Implement new serial algorithms from Parallelism TS (P0024R2)

These new (non-parallel) algorithms were added to C++17 along with the
parallel algorithms, but were missing from libstdc++.

* include/bits/algorithmfwd.h: Change title of doc group.
* include/bits/stl_algo.h (for_each_n): Add new C++17 algorithm from
P0024R2.
* include/bits/stl_numeric.h: Define doc group and add algos to it.
* include/std/numeric (__is_random_access_iter): New internal trait.
(reduce, transform_reduce, exclusive_scan, inclusive_scan)
(transform_exclusive_scan, transform_inclusive_scan): Likewise.
* testsuite/25_algorithms/for_each/for_each_n.cc: New test.
* testsuite/26_numerics/exclusive_scan/1.cc: New test.
* testsuite/26_numerics/inclusive_scan/1.cc: New test.
* testsuite/26_numerics/reduce/1.cc: New test.
* testsuite/26_numerics/transform_exclusive_scan/1.cc: New test.
* testsuite/26_numerics/transform_inclusive_scan/1.cc: New test.
* testsuite/26_numerics/transform_reduce/1.cc: New test.
* testsuite/util/testsuite_iterators.h (test_container::size()): New
member function.

From-SVN: r272459

5 years agoxtensa: fix PR target/90922
Max Filippov [Tue, 18 Jun 2019 22:19:12 +0000 (22:19 +0000)]
xtensa: fix PR target/90922

Stack pointer adjustment code in prologue missed a case of no
callee-saved registers and a stack frame size bigger than 128 bytes.
Handle that case.

This fixes the following gcc tests with call0 ABI:
  gcc.c-torture/execute/stdarg-2.c
  gcc.dg/torture/pr55882.c
  gcc.dg/torture/pr57569.c

2019-06-18  Max Filippov  <jcmvbkbc@gmail.com>
gcc/
* config/xtensa/xtensa.c (xtensa_expand_prologue): Add stack
pointer adjustment for the case of no callee-saved registers and
stack frame bigger than 128 bytes.

From-SVN: r272455

5 years ago[PR90921] Fortran OpenACC 'declare' directive's module handling causes duplicate...
Julian Brown [Tue, 18 Jun 2019 22:15:53 +0000 (00:15 +0200)]
[PR90921] Fortran OpenACC 'declare' directive's module handling causes duplicate data clauses

gcc/fortran/
PR fortran/90921
* trans-decl.c (finish_oacc_declare): Reset module_oacc_clauses
before scanning each namespace.
gcc/testsuite/
PR fortran/90921
* gfortran.dg/goacc/declare-3.f95: Update.

From-SVN: r272454

5 years ago[PR85221] Set 'omp declare target', 'omp declare target link' attributes for Fortran...
Thomas Schwinge [Tue, 18 Jun 2019 22:15:43 +0000 (00:15 +0200)]
[PR85221] Set 'omp declare target', 'omp declare target link' attributes for Fortran OpenACC 'declare'd variables

gcc/fortran/
PR fortran/85221
* trans-decl.c (add_attributes_to_decl): Handle OpenACC 'declare'
directive.
gcc/testsuite/
PR fortran/85221
* gfortran.dg/goacc/declare-3.f95: New file.

From-SVN: r272453

5 years ago[PR90859] Document status quo for "[OMP] Mappings for VLA different depending on...
Thomas Schwinge [Tue, 18 Jun 2019 22:15:16 +0000 (00:15 +0200)]
[PR90859] Document status quo for "[OMP] Mappings for VLA different depending on 'target { c && { ! lp64 } }'"

gcc/testsuite/
PR middle-end/90859
* c-c++-common/goacc/firstprivate-mappings-1.c: Update.

From-SVN: r272452

5 years agoTest cases to verify OpenACC 'firstprivate' mappings
Thomas Schwinge [Tue, 18 Jun 2019 22:15:03 +0000 (00:15 +0200)]
Test cases to verify OpenACC 'firstprivate' mappings

gcc/testsuite/
* c-c++-common/goacc/firstprivate-mappings-1.c: New file.
* g++.dg/goacc/firstprivate-mappings-1.C: Likewise.
libgomp/
* testsuite/libgomp.oacc-c++/firstprivate-mappings-1.C: New file.
* testsuite/libgomp.oacc-c-c++-common/firstprivate-mappings-1.c:
Likewise.

From-SVN: r272451

5 years agoFix description of 'GOMP_MAP_FIRSTPRIVATE'
Thomas Schwinge [Tue, 18 Jun 2019 22:14:52 +0000 (00:14 +0200)]
Fix description of 'GOMP_MAP_FIRSTPRIVATE'

..., which got garbled in r230275.

include/
* gomp-constants.h (enum gomp_map_kind): Fix description of
'GOMP_MAP_FIRSTPRIVATE'.

From-SVN: r272450

5 years agoAdd missing results check in 'libgomp.fortran/allocatable3.f90'
Thomas Schwinge [Tue, 18 Jun 2019 22:14:43 +0000 (00:14 +0200)]
Add missing results check in 'libgomp.fortran/allocatable3.f90'

libgomp/
* testsuite/libgomp.fortran/allocatable3.f90: Add missing results
check.

From-SVN: r272449

5 years agoAdd 'libgomp.oacc-fortran/allocatable-array-1.f90'
Cesar Philippidis [Tue, 18 Jun 2019 22:14:34 +0000 (15:14 -0700)]
Add 'libgomp.oacc-fortran/allocatable-array-1.f90'

libgomp/
* testsuite/libgomp.oacc-fortran/allocatable-array-1.f90: New
file.

From-SVN: r272448

5 years ago[PR90743] Fortran 'allocatable' with OpenACC data/OpenMP 'target' 'map' clauses
Thomas Schwinge [Tue, 18 Jun 2019 22:14:24 +0000 (00:14 +0200)]
[PR90743] Fortran 'allocatable' with OpenACC data/OpenMP 'target' 'map' clauses

Test what OpenMP 5.0 has to say on this topic.  And, do the same for OpenACC.

libgomp/
PR fortran/90743
* oacc-parallel.c (GOACC_parallel_keyed): Handle NULL mapping
case.
* testsuite/libgomp.fortran/target-allocatable-1-1.f90: New file.
* testsuite/libgomp.fortran/target-allocatable-1-2.f90: Likewise.
* testsuite/libgomp.oacc-fortran/allocatable-1-1.f90: Likewise.
* testsuite/libgomp.oacc-fortran/allocatable-1-2.f90: Likewise.

From-SVN: r272447

5 years ago[PR90861] Document status quo for OpenACC 'declare' not cleaning up for VLAs
Thomas Schwinge [Tue, 18 Jun 2019 22:14:14 +0000 (00:14 +0200)]
[PR90861] Document status quo for OpenACC 'declare' not cleaning up for VLAs

gcc/testsuite/
PR testsuite/90861
* c-c++-common/goacc/declare-pr90861.c: New file.
libgomp/
PR testsuite/90861
* testsuite/libgomp.oacc-c-c++-common/declare-vla.c: Update.

From-SVN: r272446

5 years ago[PR90868] Document status quo for duplicate OpenACC 'declare' directives for 'extern...
Thomas Schwinge [Tue, 18 Jun 2019 22:14:04 +0000 (00:14 +0200)]
[PR90868] Document status quo for duplicate OpenACC 'declare' directives for 'extern' variables

gcc/testsuite/
PR testsuite/90868
* c-c++-common/goacc/declare-1.c: Update.
* c-c++-common/goacc/declare-2.c: Likewise.

From-SVN: r272445

5 years ago[PR90862] OpenACC 'declare' ICE when nested inside another construct
Thomas Schwinge [Tue, 18 Jun 2019 22:13:54 +0000 (00:13 +0200)]
[PR90862] OpenACC 'declare' ICE when nested inside another construct

gcc/
PR middle-end/90862
* omp-low.c (check_omp_nesting_restrictions): Handle
GF_OMP_TARGET_KIND_OACC_DECLARE.
gcc/testsuite/
PR middle-end/90862
* c-c++-common/goacc/declare-1.c: Update.
* c-c++-common/goacc/declare-2.c: Likewise.
libgomp/
PR middle-end/90862
* testsuite/libgomp.oacc-c-c++-common/declare-1.c: Update.

From-SVN: r272444

5 years agore PR c++/84698 (ICE when using noexcept(noexcept()) declaration on global friend...
Marek Polacek [Tue, 18 Jun 2019 21:50:51 +0000 (21:50 +0000)]
re PR c++/84698 (ICE when using noexcept(noexcept()) declaration on global friend function of template class)

PR c++/84698
* g++.dg/cpp0x/noexcept42.C: New test.

From-SVN: r272443

5 years agore PR c++/71548 (Invalid declaration involving template template param causes crash)
Marek Polacek [Tue, 18 Jun 2019 20:55:46 +0000 (20:55 +0000)]
re PR c++/71548 (Invalid declaration involving template template param causes crash)

PR c++/71548
* g++.dg/cpp0x/variadic177.C: New test.

From-SVN: r272438

5 years agoi386.md (@cmp<mode>_1): Rename from cmp<mode>_1.
Uros Bizjak [Tue, 18 Jun 2019 16:55:10 +0000 (18:55 +0200)]
i386.md (@cmp<mode>_1): Rename from cmp<mode>_1.

* config/i386/i386.md (@cmp<mode>_1): Rename from cmp<mode>_1.
(@add<mode>3_carry): Rename from add<mode>3_carry.
(@sub<mode>3_carry_ccc): Rename from sub<mode>3_carry_ccc.
(@sub<mode>3_carry_ccgz): Rename form sub<mode>3_carry_ccgz.
(@copysign<mode>3_const): Rename from copysign<mode>3_const.
(@copysign<mode>3_var): Rename from copysign<mode>3_var.
(@xorsign<mode>3_1): Rename from xorsign<mode>3_1.
(@x86_shift<mode>_adj_1): Rename from x86_shift<mode>_adj_1.
(@x86_shift<mode>_adj_2): Rename from x86_shift<mode>_adj_2.
(@x86_shift<mode>_adj_3): Rename from x86_shift<mode>_adj_3.
(cmpstrnsi): Use gen_cmp_1.
(lwp_slwpcb): Use gen_lwp_slwpcb_1.
(@lwp_slwpcb<mode>_1): Rename from lwp_slwpcb<mode>_1.
(@umonitor_<mode>): Rename from umonitor_<mode>.
* config/i386/i386-expand.c (ix86_expand_copysign):
Use gen_copysign3_const and gen_copysign3_var.
(ix86_expand_xorsign): Use gen_xorsign3_1.
(ix86_expand_branch): Use gen_sub3_carry_ccc,
gen_sub3_carry_ccgz and gen_cmp1.
(ix86_expand_int_addcc): Use gen_sub3_carry and gen_add3_carry.
(ix86_split_ashl): Use gen_x86_shift_adj_1 and gen_x86_shift_adj_2.
(ix86_split_ashr): Use gen_x86_shift_adj_1 and gen_x86_shift_adj_3.
(ix86_split_lshr): Ditto.
(ix86_expand_builtin) <case IX86_BUILTIN_UMONITOR>: Use gen_umonitor.

From-SVN: r272432

5 years ago* constexpr.c (cxx_eval_store_expression): Delay target evaluation.
Jason Merrill [Tue, 18 Jun 2019 16:08:29 +0000 (12:08 -0400)]
* constexpr.c (cxx_eval_store_expression): Delay target evaluation.

From-SVN: r272431

5 years agoConsolidate constexpr array handling.
Jason Merrill [Tue, 18 Jun 2019 16:08:23 +0000 (12:08 -0400)]
Consolidate constexpr array handling.

* constexpr.c (eval_and_check_array_index): Split out from...
(cxx_eval_array_reference): ...here.
(cxx_eval_store_expression): Use it here, too.
(diag_array_subscript): Take location.  Strip location wrapper.

From-SVN: r272430

5 years agoHandle constexpr conversion from and then to the same type.
Jason Merrill [Tue, 18 Jun 2019 16:08:17 +0000 (12:08 -0400)]
Handle constexpr conversion from and then to the same type.

* constexpr.c (cxx_eval_constant_expression): Handle conversion from
and then to the same type.

From-SVN: r272429

5 years agotree.c (build_constructor): Add MEM_STAT_DECL.
Jason Merrill [Tue, 18 Jun 2019 16:08:12 +0000 (12:08 -0400)]
tree.c (build_constructor): Add MEM_STAT_DECL.

* tree.c (build_constructor): Add MEM_STAT_DECL.

gcc/cp/
* constexpr.c (unshare_constructor): Add MEM_STAT_DECL.

From-SVN: r272428

5 years ago[AArch64] Add a new CC mode for SVE conditions
Richard Sandiford [Tue, 18 Jun 2019 14:54:34 +0000 (14:54 +0000)]
[AArch64] Add a new CC mode for SVE conditions

The SVE ACLE patches need to introduce a new CC_NZC mode for the
conditions that can be tested after a PTRUE.  In particular, LT needs
to map to "mi"/"first" and GE to "pl"/"nfrst", instead of the normal
CC mapping.

Another advantage of using a separate mode is that we can print the SVE
names of the conditions, which makes the output a bit easier to read.
It therefore seems like an independent improvement that can go in now.

The patch also avoids using (compare X (const_int 0)), because that gets
folded away when used with LTU and GEU ("cc"/"last" and "cs"/"nlast").
Just using an unspec should be OK.

The full set of conditions can't be tested without other SVE ACLE patches.

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* config/aarch64/aarch64-modes.def (CC_NZC): New CC_MODE.
* config/aarch64/aarch64-sve.md (*<optab><mode>3_cc)
(ptest_ptrue<mode>, while_ult<GPI:mode><PRED_ALL:mode>)
(*while_ult<GPI:mode><PRED_ALL:mode>_cc, *cmp<cmp_op><mode>)
(*cmp<cmp_op><mode>_ptest, *cmp<cmp_op><mode>_cc)
(*pred_cmp<cmp_op><mode>_combine, *pred_cmp<cmp_op><mode>)
(vec_cmp<mode><vpred>, vec_cmpu<mode><vpred>, cbranch<mode>4):
Use CC_NZC instead of CC.
* config/aarch64/aarch64.md (condjump): Print a '.' in SVE conditions.
* config/aarch64/aarch64.c (aarch64_sve_condition_codes): New variable.
(aarch64_print_operand): Handle E_CC_NZCmode.
(aarch64_emit_sve_ptrue_op_cc): Use gen_set_clobber_cc_nzc instead
of gen_set_clobber_cc.

gcc/testsuite/
* gcc.target/aarch64/sve/struct_vect_18.c: Allow branches to
contain dots.
* gcc.target/aarch64/sve/struct_vect_19.c: Likewise.
* gcc.target/aarch64/sve/struct_vect_20.c: Likewise.
* gcc.target/aarch64/sve/struct_vect_21.c: Likewise.
* gcc.target/aarch64/sve/struct_vect_22.c: Likewise.
* gcc.target/aarch64/sve/struct_vect_23.c: Likewise.
* gcc.target/aarch64/sve/unroll-1.c: Likewise.
* gcc.target/aarch64/sve/while_1.c: Check for b.any.

From-SVN: r272427

5 years ago[AArch64] Tabify aarch64-sve.md
Richard Sandiford [Tue, 18 Jun 2019 14:52:41 +0000 (14:52 +0000)]
[AArch64] Tabify aarch64-sve.md

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* config/aarch64/aarch64-sve.md: Tabify file.

From-SVN: r272426

5 years ago[AArch64] Factor out pfalse predicate creation
Richard Sandiford [Tue, 18 Jun 2019 14:51:02 +0000 (14:51 +0000)]
[AArch64] Factor out pfalse predicate creation

Following on from the previous ptrue patch.

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* config/aarch64/aarch64-protos.h (aarch64_pfalse_reg): Declare.
* config/aarch64/aarch64.c (aarch64_pfalse_reg): New function.
* config/aarch64/aarch64-sve.md: Use it.

From-SVN: r272425

5 years ago[AArch64] Factor out ptrue predicate creation
Richard Sandiford [Tue, 18 Jun 2019 14:49:28 +0000 (14:49 +0000)]
[AArch64] Factor out ptrue predicate creation

This is the first step to canonicalising predicate constants so that
they can be reused between modes.

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* config/aarch64/aarch64-protos.h (aarch64_ptrue_reg): Declare.
* config/aarch64/aarch64.c (aarch64_ptrue_reg): New functions.
(aarch64_expand_sve_widened_duplicate, aarch64_expand_sve_mem_move)
(aarch64_maybe_expand_sve_subreg_move, aarch64_evpc_rev_local)
(aarch64_expand_sve_vec_cmp_int): Use it.
(aarch64_expand_sve_vec_cmp_float): Likewise.
* config/aarch64/aarch64-sve.md: Likewise throughout.

From-SVN: r272424

5 years ago[AArch64] Simplify SVE IFN_COND patterns
Richard Sandiford [Tue, 18 Jun 2019 14:47:42 +0000 (14:47 +0000)]
[AArch64] Simplify SVE IFN_COND patterns

This patch makes the binary IFN_COND patterns use the same approach
as the ternary patterns, with one pattern handling the cases in
which the "else" value isn't tied to one of the other inputs.

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>
    Kugan Vivekanandarajah  <kuganv@linaro.org>

gcc/
* config/aarch64/aarch64-sve.md (*cond_<optab><mode>_0): Delete.
(*cond_<optab><mode>_z): Fold into...
(*cond_<optab><mode>_any): ...here.  Also handle cases in which
operand 4 can be tied to operand 0 (either inherently or via RA).

From-SVN: r272423

5 years agopr81563.c (dg-final): Check that no registers are restored from %esp.
Uros Bizjak [Tue, 18 Jun 2019 14:31:11 +0000 (16:31 +0200)]
pr81563.c (dg-final): Check that no registers are restored from %esp.

* gcc.target/i386/pr81563.c (dg-final): Check that no
registers are restored from %esp.

From-SVN: r272422

5 years agore PR debug/90900 (ICE in copy_rtx, at rtl.c:376)
Richard Biener [Tue, 18 Jun 2019 13:56:24 +0000 (13:56 +0000)]
re PR debug/90900 (ICE in copy_rtx, at rtl.c:376)

2019-06-18  Richard Biener  <rguenther@suse.de>

PR debug/90900
* cfgexpand.c (expand_debug_expr): Treat NOTE_P DECL_RTL
as if optimized away.

* gcc.dg/gomp/pr90900.c: New testcase.

From-SVN: r272421

5 years agopr90866-2.c: Remove a pointless declaration to avoid compilation errors on arm-none...
Martin Sebor [Tue, 18 Jun 2019 13:06:31 +0000 (13:06 +0000)]
pr90866-2.c: Remove a pointless declaration to avoid compilation errors on arm-none-eabi.

gcc/testsuite/ChangeLog:

* gcc.dg/pr90866-2.c: Remove a pointless declaration
to avoid compilation errors on arm-none-eabi.

From-SVN: r272418

5 years agore PR ada/80590 (non-bootstrap build failure of Ada runtime)
Arnaud Charlet [Tue, 18 Jun 2019 11:45:37 +0000 (11:45 +0000)]
re PR ada/80590 (non-bootstrap build failure of Ada runtime)

PR ada/80590

* sem_ch5.adb (Analyze_Loop_Statement): Avoid exception propagation
during normal processing.

From-SVN: r272417

5 years agoAvoid undefined behaviour in std::byte operators (LWG 2950)
Jonathan Wakely [Tue, 18 Jun 2019 11:39:43 +0000 (12:39 +0100)]
Avoid undefined behaviour in std::byte operators (LWG 2950)

* include/c_global/cstddef (std::byte): Perform arithmetic operations
in unsigned int to avoid promotion (LWG 2950).

From-SVN: r272415

5 years agoaarch64: fix gcc.target/aarch64/pcs_attribute-2.c on non-gnu targets
Szabolcs Nagy [Tue, 18 Jun 2019 11:11:07 +0000 (11:11 +0000)]
aarch64: fix gcc.target/aarch64/pcs_attribute-2.c on non-gnu targets

Move the ifunc symbol tests into a separate file with dg-require-ifunc.
And added a base pcs ifunc symbol to the test for completeness.

gcc/testsuite/ChangeLog:

* gcc.target/aarch64/pcs_attribute-2.c: Remove ifunc usage.
* gcc.target/aarch64/pcs_attribute-3.c: New test.

From-SVN: r272414

5 years ago[nvptx] Use define_insn parametrization
Tom de Vries [Tue, 18 Jun 2019 09:19:41 +0000 (09:19 +0000)]
[nvptx] Use define_insn parametrization

Parametrize some define_insn to simplify code in define_expands generating
those insns.

Build and reg-tested on x86_64 with nvptx accelerator.

2019-06-18  Tom de Vries  <tdevries@suse.de>

* config/nvptx/nvptx-protos.h (gen_set_softstack_insn): Remove.
* config/nvptx/nvptx.c (gen_set_softstack_insn): Remove.
* config/nvptx/nvptx.md (define_insn "set_softstack_<mode>"): Rename to ...
(define_insn "@set_softstack_<mode>"): ... this.
(define_insn "omp_simt_enter_<mode>"): Rename to ...
(define_insn "@omp_simt_enter_<mode>"): ... this.
(define_insn "omp_simt_exit_<mode>"): Rename to ...
(define_insn "@omp_simt_exit_<mode>"): ... this.

From-SVN: r272413

5 years ago[nvptx] Fix __main missing prototype warning in crt0.c
Tom de Vries [Tue, 18 Jun 2019 09:19:28 +0000 (09:19 +0000)]
[nvptx] Fix __main missing prototype warning in crt0.c

Atm we see:
...
libgcc/config/nvptx/crt0.c:36:1: warning: no previous prototype for \
  ‘__main’ [-Wmissing-prototypes]
...

Fix this by adding the prototype.

Build and reg-tested on nvptx.
Build and reg-tested on x86_64 with nvptx accelerator.

2019-06-18  Tom de Vries  <tdevries@suse.de>

* config/nvptx/crt0.c (__main): Declare.

From-SVN: r272412

5 years agoRestore correct iv step for fully-masked loops
Richard Sandiford [Tue, 18 Jun 2019 09:18:17 +0000 (09:18 +0000)]
Restore correct iv step for fully-masked loops

r272233 introduced a large number of execution failures on SVE.
The patch hard-coded an IV step of VF, but for SLP groups it needs
to be VF * group size.

Also, iv_precision had type widest_int but only needs to be unsigned int.

2019-06-18  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* tree-vect-loop-manip.c (vect_set_loop_masks_directly): Remove
vf parameter.  Restore the previous iv step of nscalars_step,
but give it iv_type rather than compare_type.  Tweak code order
to match the comments.
(vect_set_loop_condition_masked): Update accordingly.
* tree-vect-loop.c (vect_verify_full_masking): Use "unsigned int"
for iv_precision.  Tweak comment formatting.

From-SVN: r272411

5 years agoDarwin - strip trailing whitespace from a file, NFC.
Iain Sandoe [Tue, 18 Jun 2019 08:49:40 +0000 (08:49 +0000)]
Darwin - strip trailing whitespace from a file, NFC.

2019-06-18  Iain Sandoe  <iain@sandoe.co.uk>

        * config/darwin.c: Strip trailing whitespace.

From-SVN: r272410

5 years ago[Darwin] The need for FDE symbols is dependent on linker used, not OS rev.
Iain Sandoe [Tue, 18 Jun 2019 08:41:12 +0000 (08:41 +0000)]
[Darwin] The need for FDE symbols is dependent on linker used, not OS rev.

For very old toolchains, the compiler generated extra symbols that mark the
start of each FDE.  We no longer need this (since xcode 3 era).

Since we have detection of the linker version, we can use that directly to
determine if support is needed.

2019-06-18  Iain Sandoe  <iain@sandoe.co.uk>

* config/darwin.c (darwin_emit_unwind_label): New default to false.
(darwin_override_options): Set darwin_emit_unwind_label as needed.

From-SVN: r272409

5 years agoFix IPA-CP topological sorting
Martin Jambor [Tue, 18 Jun 2019 08:14:09 +0000 (10:14 +0200)]
Fix IPA-CP topological sorting

2019-06-18  Martin Jambor  <mjambor@suse.cz>

PR ipa/90889
* ipa-cp.c (ignore_edge_p): Do not ignore edges when only the
caller does not have flag_ipa_cp set.

From-SVN: r272408

5 years ago[Vectorizer] Support masking fold left reductions
Alejandro Martinez [Tue, 18 Jun 2019 08:09:00 +0000 (08:09 +0000)]
[Vectorizer] Support masking fold left reductions

This patch adds support in the vectorizer for masking fold left reductions.
This avoids the need to insert a conditional assignement with some identity
value.

From-SVN: r272407

5 years agolibsanitizer - remove reference to __cxa_rethrow_primary_exception.
Iain Sandoe [Tue, 18 Jun 2019 08:01:00 +0000 (08:01 +0000)]
libsanitizer - remove reference to __cxa_rethrow_primary_exception.

For some Darwin versions the absence of the rethrow_primary_exception
symbol causes almost all sanitizer tests to fail.
The symbol is not present in libstdc++ and, therefore is is correct to
remove the reference to it for all platforms.  We do this by adding a
new guard "ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION".

libsanitizer/

2019-06-18  Iain Sandoe  <iain@sandoe.co.uk>

PR libsanitizer/87880
* asan/asan_interceptors.h:
(ASAN_INTERCEPT___CXA_RETHROW_PRIMARY_EXCEPTION): New.
* asan/Makefile.am (DEFS): Add
ASAN_HAS_CXA_RETHROW_PRIMARY_EXCEPTION, defined to 0.
* asan/Makefile.in: Regenerated.
* asan/libtool-version: Bump version.

From-SVN: r272406

5 years agoAdd predict_doloop_p target hook
Kewen Lin [Tue, 18 Jun 2019 05:08:02 +0000 (05:08 +0000)]
Add predict_doloop_p target hook

    Add one target hook predict_doloop_p, it return true if we can predict it
    is possible to use a low-overhead loop, it can help ivopts to make some
    better decisions.

    PR middle-end/80791
    * target.def (predict_doloop_p): New hook.
    * targhooks.h (default_predict_doloop_p): New declaration.
    * targhooks.c (default_predict_doloop_p): New function.
    * doc/tm.texi.in (TARGET_PREDICT_DOLOOP_P): New hook.
    * doc/tm.texi: Regenerate.
    * config/rs6000/rs6000.c (rs6000_predict_doloop_p): New function.
    (TARGET_PREDICT_DOLOOP_P): New macro.
    * tree-ssa-loop-ivopts.c (generic_predict_doloop_p): New function.

From-SVN: r272405

5 years agoDaily bump.
GCC Administrator [Tue, 18 Jun 2019 00:16:21 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r272404

5 years agoomp-low.c (struct omp_context): Add scan_inclusive field.
Jakub Jelinek [Mon, 17 Jun 2019 21:20:00 +0000 (23:20 +0200)]
omp-low.c (struct omp_context): Add scan_inclusive field.

* omp-low.c (struct omp_context): Add scan_inclusive field.
(scan_omp_1_stmt) <case GIMPLE_OMP_SCAN>: Set ctx->scan_inclusive
if inclusive scan.
(struct omplow_simd_context): Add lastlane member.
(lower_rec_simd_input_clauses): Add rvar argument, handle inscan
reductions.  Build 2 or 3 argument .GOMP_SIMD_LANE calls rather than
1 or 2 argument.
(lower_rec_input_clauses): Handle inscan reductions in simd contexts.
(lower_lastprivate_clauses): Set TREE_THIS_NOTRAP on the ARRAY_REF.
(lower_omp_scan): New function.
(lower_omp_1) <case GIMPLE_OMP_SCAN>: Use lower_omp_scan.
* tree-ssa-dce.c (eliminate_unnecessary_stmts): For IFN_GOMP_SIMD_LANE
check 3rd argument if present rather than 2nd.
* tree-vectorizer.h (struct _loop_vec_info): Add scan_map member.
(struct _stmt_vec_info): Change simd_lane_access_p from bool into
2-bit bitfield.
* tree-vect-loop.c (_loop_vec_info::_loop_vec_info): Initialize
scan_map.  For IFN_GOMP_SIMD_LANE check 3rd argument if present rather
than 2nd.
(_loop_vec_info::~_loop_vec_info): Delete scan_map.
* tree-vect-data-refs.c (vect_analyze_data_ref_accesses): Allow two
different STMT_VINFO_SIMD_LANE_ACCESS_P refs if they have the same
init.
(vect_find_stmt_data_reference): Encode in ->aux the 2nd
IFN_GOMP_SIMD_LANE argument.
(vect_analyze_data_refs): Set STMT_VINFO_SIMD_LANE_ACCESS_P from the
encoded ->aux value.
* tree-vect-stmts.c: Include attribs.h.
(vectorizable_call): Adjust comment about IFN_GOMP_SIMD_LANE.
(scan_operand_equal_p, check_scan_store, vectorizable_scan_store): New
functions.
(vectorizable_load): For STMT_VINFO_SIMD_LANE_ACCESS_P tests use != 0.
(vectorizable_store): Handle STMT_VINFO_SIMD_LANE_ACCESS_P > 1.
cp/
* semantics.c (finish_omp_clauses): For OMP_CLAUSE_REDUCTION_INSCAN
set need_copy_assignment.
testsuite/
* gcc.dg/vect/vect-simd-8.c: New test.
* gcc.dg/vect/vect-simd-9.c: New test.
* g++.dg/vect/simd-2.cc: New test.
* g++.dg/gomp/scan-1.C: New test.

From-SVN: r272399

5 years agore PR target/62055 (missed optimization: recognize fnabs (FP negative absolute value...
Uros Bizjak [Mon, 17 Jun 2019 18:40:22 +0000 (20:40 +0200)]
re PR target/62055 (missed optimization: recognize fnabs (FP negative absolute value) (x86-64))

PR target/62055
* config/i386/i386.md (*nabstf2_1): New insn pattern.
(*nabs<mode>2_1): Ditto.
(nabs sse-reg splitter): New splitter.
* config/i386/sse.md (*nabs<mode>2): New insn_and_split pattern.

testsuite/ChangeLog:

PR target/62055
* gcc.target/i386/fnabs.c: New test.

From-SVN: r272396

5 years agoPR c++/83820 - excessive attribute arguments not detected.
Marek Polacek [Mon, 17 Jun 2019 18:26:07 +0000 (18:26 +0000)]
PR c++/83820 - excessive attribute arguments not detected.

* parser.c (cp_parser_std_attribute): Detect excessive arguments.

* g++.dg/cpp0x/gen-attrs-67.C: New test.

From-SVN: r272395

5 years agoFix AIX test failure due to replacement operator delete
Jonathan Wakely [Mon, 17 Jun 2019 15:51:31 +0000 (16:51 +0100)]
Fix AIX test failure due to replacement operator delete

On AIX the sized delete defined in the library will call the non-sized
delete defined in the library, not the replacement version defined in
the test file. By also replacing sized delete we make the test pass
everywhere.

* testsuite/20_util/allocator/1.cc: Add sized delete, which fixes a
failure on AIX.

From-SVN: r272391

5 years agore PR bootstrap/90873 (-Wmaybe-uninitialized warning in gcc/tree-ssa-forwprop.c break...
Jan Hubicka [Mon, 17 Jun 2019 15:43:23 +0000 (17:43 +0200)]
re PR bootstrap/90873 (-Wmaybe-uninitialized warning in gcc/tree-ssa-forwprop.c breaks 32-bit bootstrap)

PR bootstrap/90873.
* tree-ssa-alias.c (indirect_ref_may_alias_decl_p): Fix
TMR index check.

From-SVN: r272390

5 years ago[nvptx] Fix missing mode warnings in nvptx.md, omp part
Tom de Vries [Mon, 17 Jun 2019 14:50:21 +0000 (14:50 +0000)]
[nvptx] Fix missing mode warnings in nvptx.md, omp part

Fix these warnings:
...
gcc/config/nvptx/nvptx.md:1074:1: warning: operand 0 missing mode?
gcc/config/nvptx/nvptx.md:1240:1: warning: operand 0 missing mode?
gcc/config/nvptx/nvptx.md:1240:1: warning: operand 1 missing mode?
gcc/config/nvptx/nvptx.md:1240:1: warning: operand 2 missing mode?
gcc/config/nvptx/nvptx.md:1268:1: warning: operand 0 missing mode?
...

Build and reg-tested on x86_64 with nvptx accelerator.

2019-06-17  Tom de Vries  <tdevries@suse.de>

* config/nvptx/nvptx-protos.h (gen_set_softstack_insn): Declare.
* config/nvptx/nvptx.c (gen_set_softstack_insn): New function.
* config/nvptx/nvptx.md (define_insn "set_softstack_insn"): Rename to
...
(define_insn "set_softstack_<mode>"): ... this.  Use P iterator on
match_operand 0.
(define_insn "omp_simt_enter_insn"): Rename to ...
(define_insn "omp_simt_enter_<mode>"): ... this.  Use P iterator on
match_operand 0, 1 and 2, as well as the unspec_volatile result.
(define_expand "omp_simt_enter): Use gen_omp_simt_enter_di and
gen_omp_simt_enter_si.
(define_expand "omp_simt_exit"): New.
(define_insn "omp_simt_exit"): Rename to ...
(define_insn "omp_simt_exit_<mode>"): ... this.  Use P iterator on
match_operand 0.

From-SVN: r272388

5 years ago[netbsd][aarch64] add netbsd/aarch64 target
Matthew Green [Mon, 17 Jun 2019 14:43:11 +0000 (14:43 +0000)]
[netbsd][aarch64] add netbsd/aarch64 target

Initial support for AArch64 running NetBSD.

Matthew Green <mrg@eterna.com.au>
Maya Rashish <coypu@sdf.org>

gcc:
* config.gcc (aarch64*-*-netbsd*): New target.
* config/aarch64/aarch64-netbsd.h: New file.
* config/aarch64/t-aarch64-netbsd: Likewise.

libgcc:
* config.host (aarch64*-*-netbsd*): New case.

Co-Authored-By: Maya Rashish <coypu@sdf.org>
From-SVN: r272387

5 years agoAdd 'noexcept' to std::lerp
Jonathan Wakely [Mon, 17 Jun 2019 14:32:44 +0000 (15:32 +0100)]
Add 'noexcept' to std::lerp

* include/c_global/cmath (__lerp, lerp): Add noexcept (LWG 3201).

From-SVN: r272386

5 years agoPR libstdc++/90281 Fix string conversions for filesystem::path
Jonathan Wakely [Mon, 17 Jun 2019 14:19:04 +0000 (15:19 +0100)]
PR libstdc++/90281 Fix string conversions for filesystem::path

Fix several bugs in the encoding conversions for filesystem::path that
prevent conversion of Unicode characters outside the Basic Multilingual
Plane, and prevent returning basic_string specializations with
alternative allocator types.

The std::codecvt_utf8 class template is not suitable for UTF-16
conversions because it uses UCS-2 instead. For conversions between UTF-8
and UTF-16 either std::codecvt<C, char, mbstate> or
codecvt_utf8_utf16<C> must be used.

The __str_codecvt_in and __str_codecvt_out utilities do not
return false on a partial conversion (e.g. for invalid or incomplete
Unicode input). Add new helpers that treat partial conversions as
errors, and use them for all filesystem::path conversions.

PR libstdc++/90281 Fix string conversions for filesystem::path
* include/bits/fs_path.h (u8path) [_GLIBCXX_FILESYSTEM_IS_WINDOWS]:
Use codecvt_utf8_utf16 instead of codecvt_utf8. Use
__str_codecvt_in_all to fail for partial conversions and throw on
error.
[!_GLIBCXX_FILESYSTEM_IS_WINDOWS && _GLIBCXX_USE_CHAR8_T]
(path::_Cvt<char8_t>): Add explicit specialization.
[_GLIBCXX_FILESYSTEM_IS_WINDOWS] (path::_Cvt::_S_wconvert): Remove
overloads.
[_GLIBCXX_FILESYSTEM_IS_WINDOWS] (path::_Cvt::_S_convert): Use
if-constexpr instead of dispatching to _S_wconvert. Use codecvt
instead of codecvt_utf8. Use __str_codecvt_in_all and
__str_codecvt_out_all.
[!_GLIBCXX_FILESYSTEM_IS_WINDOWS] (path::_Cvt::_S_convert): Use
codecvt instead of codecvt_utf8. Use __str_codecvt_out_all.
(path::_S_str_convert) [_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use
codecvt_utf8_utf16 instead of codecvt_utf8. Construct return values
with allocator. Use __str_codecvt_out_all. Fallthrough to POSIX code
after converting to UTF-8.
(path::_S_str_convert): Use codecvt instead of codecvt_utf8. Use
__str_codecvt_in_all.
(path::string): Fix initialization of string types with different
allocators.
(path::u8string) [_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use
codecvt_utf8_utf16 instead of codecvt_utf8. Use __str_codecvt_out_all.
* include/bits/locale_conv.h (__do_str_codecvt): Reorder static and
runtime conditions.
(__str_codecvt_out_all, __str_codecvt_in_all): New functions that
return false for partial conversions.
* include/experimental/bits/fs_path.h (u8path):
[_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Implement correctly for mingw.
[_GLIBCXX_FILESYSTEM_IS_WINDOWS] (path::_Cvt::_S_wconvert): Add
missing handling for char8_t. Use codecvt and codecvt_utf8_utf16
instead of codecvt_utf8. Use __str_codecvt_in_all and
__str_codecvt_out_all.
[!_GLIBCXX_FILESYSTEM_IS_WINDOWS] (path::_Cvt::_S_convert): Use
codecvt instead of codecvt_utf8. Use __str_codecvt_out_all.
(path::string) [_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use
codecvt_utf8_utf16 instead of codecvt_utf8. Construct return values
with allocator. Use __str_codecvt_out_all and __str_codecvt_in_all.
(path::string) [!_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use
__str_codecvt_in_all.
(path::u8string) [_GLIBCXX_FILESYSTEM_IS_WINDOWS]: Use
codecvt_utf8_utf16 instead of codecvt_utf8. Use __str_codecvt_out_all.
* src/c++17/fs_path.cc (path::_S_convert_loc): Use
__str_codecvt_in_all.
* src/filesystem/path.cc (path::_S_convert_loc): Likewise.
* testsuite/27_io/filesystem/path/construct/90281.cc: New test.
* testsuite/27_io/filesystem/path/factory/u8path.cc: New test.
* testsuite/27_io/filesystem/path/native/string.cc: Test with empty
strings and with Unicode characters outside the basic multilingual
plane.
* testsuite/27_io/filesystem/path/native/alloc.cc: New test.
* testsuite/experimental/filesystem/path/construct/90281.cc: New test.
* testsuite/experimental/filesystem/path/factory/u8path.cc: New test.
* testsuite/experimental/filesystem/path/native/alloc.cc: New test.
* testsuite/experimental/filesystem/path/native/string.cc: Test with
empty strings and with Unicode characters outside the basic
multilingual plane.

From-SVN: r272385

5 years ago[PR c++/90754] name lookup ICE
Nathan Sidwell [Mon, 17 Jun 2019 13:46:59 +0000 (13:46 +0000)]
[PR c++/90754] name lookup ICE

https://gcc.gnu.org/ml/gcc-patches/2019-06/msg00952.html
PR c++/90754
* name-lookup.c (lookup_type_scope_1): Calll qualify_lookup before
checking context.

PR c++/90754
* g++.dg/lookup/pr90754.C: New.

From-SVN: r272384

5 years agotree-ssa-alias.c (aliasing_component_refs_p): Consider only the access path from...
Jan Hubicka [Mon, 17 Jun 2019 13:44:58 +0000 (15:44 +0200)]
tree-ssa-alias.c (aliasing_component_refs_p): Consider only the access path from base to first VIEW_CONVERT_EXPR or BIT_FIELD_REF.

* tree-ssa-alias.c (aliasing_component_refs_p): Consider only
the access path from base to first VIEW_CONVERT_EXPR or
BIT_FIELD_REF.

From-SVN: r272383

5 years agoImprove PR64242 testcase
Wilco Dijkstra [Mon, 17 Jun 2019 11:25:12 +0000 (11:25 +0000)]
Improve PR64242 testcase

Clear the input array to avoid the testcase accidentally
passing with an incorrect frame pointer.

Committed as obvious.

testsuite/
PR middle-end/64242
* gcc.c-torture/execute/pr64242.c: Improve test.

From-SVN: r272382

5 years agoSimplify node ownership in _Hashtable members
François Dumont [Mon, 17 Jun 2019 10:25:04 +0000 (10:25 +0000)]
Simplify node ownership in _Hashtable members

Introduce an RAII type to manage nodes in unordered containers while
they are being inserted. If the caller always owns a node until it is
inserted, then the insertion functions don't need to deallocate on
failure. This allows a FIXME in the node re-insertion API to be removed.

Also change extract(const key_type&) to not call extract(const_iterator)
anymore.  This avoids looping through the bucket nodes again to find the
node before the one being extracted.

2019-06-17  François Dumont  <fdumont@gcc.gnu.org>
    Jonathan Wakely  <jwakely@redhat.com>

* include/bits/hashtable.h (struct _Hashtable::_Scoped_node): New type.
(_Hashtable::_M_insert_unique_node): Add key_type parameter. Don't
deallocate node if insertion fails.
(_Hashtable::_M_insert_multi_node): Likewise.
(_Hashtable::_M_reinsert_node): Pass additional key argument.
(_Hashtable::_M_reinsert_node_multi): Likewise. Remove FIXME.
(_Hashtable::_M_extract_node(size_t, __node_base*)): New function.
(_Hashtable::extract(const_iterator)): Use _M_extract_node.
(_Hashtable::extract(const _Key&)): Likewise.
(_Hashtable::_M_merge_unique): Pass additional key argument.
(_Hashtable::_M_emplace<Args>(true_type, Args&&...)): Likewise. Use
_Scoped_node.
(_Hashtable::_M_insert): Likewise.
* include/bits/hashtable_policy.h (_Map_base::operator[]): Likewise.
(_Hashtable_alloc): Add comments to functions with misleading names.

Co-Authored-By: Jonathan Wakely <jwakely@redhat.com>
From-SVN: r272381

5 years agotree-ssa-alias.c (nonoverlapping_component_refs_p): Also truncate access path on...
Jan Hubicka [Mon, 17 Jun 2019 10:22:39 +0000 (12:22 +0200)]
tree-ssa-alias.c (nonoverlapping_component_refs_p): Also truncate access path on BIT_FIELD_REFs.

* tree-ssa-alias.c (nonoverlapping_component_refs_p): Also truncate
access path on BIT_FIELD_REFs.

From-SVN: r272380

5 years agoRemove dead code (PR ipa/90874).
Martin Liska [Mon, 17 Jun 2019 09:03:26 +0000 (11:03 +0200)]
Remove dead code (PR ipa/90874).

2019-06-17  Martin Liska  <mliska@suse.cz>

PR ipa/90874
* ipa-utils.h (odr_type_p): Remove dead code.

From-SVN: r272378

5 years agoRemove support for alternative Solaris 11.4 ld -V output
Rainer Orth [Mon, 17 Jun 2019 08:30:16 +0000 (08:30 +0000)]
Remove support for alternative Solaris 11.4 ld -V output

* configure.ac (ld_vers) <*-*-solaris2*>: Remove support for
alternative Solaris 11.4 format.
* configure: Regenerate.

From-SVN: r272377

5 years agoFix tests that fail without PCH
Jonathan Wakely [Mon, 17 Jun 2019 08:18:17 +0000 (09:18 +0100)]
Fix tests that fail without PCH

The recent change to stop transitively including <string> broke some
tests, but only when the library is configured without PCH, because
otherwise the <string> header still gets included via the precompiled
<bits/stdc++.h> header.

* testsuite/20_util/bad_function_call/what.cc: Include <string> header
for std::string.
* testsuite/20_util/shared_ptr/cons/weak_ptr_expired.cc: Likewise.
* testsuite/20_util/tuple/cons/allocator_with_any.cc: Include <memory>
header for std::allocator.
* testsuite/23_containers/array/tuple_interface/tuple_element.cc: Add
using-declaration for std::size_t.
* testsuite/23_containers/array/tuple_interface/tuple_size.cc:
Likewise.
* testsuite/23_containers/deque/cons/55977.cc: Include <istream> for
std::istream.
* testsuite/23_containers/vector/cons/55977.cc: Likewise.
* testsuite/experimental/map/erasure.cc: Include <string> for
std::string.
* testsuite/experimental/unordered_map/erasure.cc: Likewise.

From-SVN: r272376