1. 05 Jun, 2018 15 commits
    • re PR c++/85976 (ICE in cp_tree_equal when building Blitz. May be nested templates.) · 05f9a806
      	PR c++/85976
      	* tree.c (cp_tree_equal): Handle USING_DECL.
      
      	* g++.dg/cpp0x/alias-decl-64.C: New test.
      
      From-SVN: r261200
      Marek Polacek committed
    • Support variables in expansion of -fprofile-generate option (PR gcov-profile/47618). · 97a53d1d
      2018-06-05  Martin Liska  <mliska@suse.cz>
      
      	PR gcov-profile/47618
      	* doc/invoke.texi: Document how -fprofile-dir format
              is extended.
      2018-06-05  Martin Liska  <mliska@suse.cz>
      
      	PR gcov-profile/47618
      	* libgcov-driver-system.c (replace_filename_variables): New
              function.
      	(gcov_exit_open_gcda_file): Use it.
      
      From-SVN: r261199
      Martin Liska committed
    • PR c++/85731 - wrong error with qualified-id in template. · 066c4268
      	* semantics.c (finish_qualified_id_expr): build_qualified_name
      	for unbound names in the current class.
      
      From-SVN: r261196
      Jason Merrill committed
    • tree-cfgcleanup.c (cleanup_control_flow_pre): For edge removal pretend DOM info… · b401e50f
      tree-cfgcleanup.c (cleanup_control_flow_pre): For edge removal pretend DOM info isn't available so we do not update it and...
      
      2018-06-05  Richard Biener  <rguenther@suse.de>
      
      	* tree-cfgcleanup.c (cleanup_control_flow_pre): For edge
      	removal pretend DOM info isn't available so we do not update
      	it and only remove edges, not dominated blocks.  Actually free
      	DOM info in case we removed something.  Remove unreachable blocks.
      	(mfb_keep_latches): Work with either DOM info or marked backedges.
      	(cleanup_tree_cfg_noloop): Do not remove unreachable blocks
      	first.  Mark backedges if DOM info isn't available.
      	(Re-)compute DOM info after cleanup_control_flow_pre.
      
      From-SVN: r261195
      Richard Biener committed
    • tree-cfg.c (struct locus_discrim_map): Store line, not location. · 62761a0f
      2018-06-05  Richard Biener  <rguenther@suse.de>
      
      	* tree-cfg.c (struct locus_discrim_map): Store line, not location.
      	(locus_discrim_hasher::hash): Adjust.
      	(locus_discrim_hasher::equal): Likewise.
      	(next_discriminator_for_locus): Work on line directly.
      	(same_line_p): Pass in expanded locus1 as well.
      	(assign_discriminators): Avoid redundant location expansions.
      
      From-SVN: r261194
      Richard Biener committed
    • re PR c/86046 (ICE in execute_todo, at passes.c:2043) · fa385ff4
      2018-06-05  Richard Biener  <rguenther@suse.de>
      
      	PR tree-optimization/86046
      	* tree-ssa.c (maybe_optimize_var): Clear DECL_GIMPLE_REG_P
      	if required after clearing TREE_ADDRESSABLE.
      
      	* gcc.dg/pr86046.c: New testcase.
      
      From-SVN: r261193
      Richard Biener committed
    • re PR tree-optimization/86047 (ICE in for_each_index, at tree-ssa-loop.c:647) · d12fd774
      2018-06-05  Richard Biener  <rguenther@suse.de>
      
      	PR tree-optimization/86047
      	* tree-ssa-loop.c (for_each_index): Glob handling of all
      	decls and constants and really handle all of them.
      
      	* gcc.dg/pr86047.c: New testcase.
      
      From-SVN: r261192
      Richard Biener committed
    • [arm] PR target/81497: Fix arm_acle.h for C++ · 12b2b910
      When trying to compile something with arm_acle.h using G++ we get a number of nasty errors:
      arm_acle.h:48:49: error: invalid conversion from ‘const void*’ to ‘const int*’ [-fpermissive]
          return __builtin_arm_ldc (__coproc, __CRd, __p);
      
      This is because the intrinsics that are supposed to be void return the "result" of their builtin,
      which is void. C lets that slide but C++ complains.
      
      After fixing that we run into further errors:
      arm_acle.h:48:46: error: invalid conversion from 'const void*' to 'const int*' [-fpermissive]
          return __builtin_arm_ldc (__coproc, __CRd, __p);
                                                     ^~~
      Because the pointer arguments in these intrinsics are void pointers but the builtin
      expects int pointers. So this patch introduces new qualifiers for void pointers and their
      const-qualified versions and uses that in the specification of these intrinsics.
      
      This gives us the opportunity of creating an arm subdirectory in g++.dg and inaugurates it
      with the first arm-specific C++ tests (in that directory).
      
      
      	PR target/81497
      	* config/arm/arm-builtins.c (arm_type_qualifiers): Add
      	qualifier_void_pointer and qualifier_const_void_pointer.
      	(arm_ldc_qualifiers, arm_stc_qualifiers): Use the above.
      	(arm_init_builtins): Handle the above.
      	* config/arm/arm_acle.h (__arm_cdp, __arm_ldc, __arm_ldcl, __arm_stc,
      	__arm_stcl, __arm_mcr, __arm_cdp2, __arm_ldc2, __arm_ldcl2, __arm_stc2,
      	__arm_stcl2,__arm_mcr2, __arm_mcrr, __arm_mcrr2): Remove return for
      	void intrinsics.
      
      	* g++.target/arm/arm.exp: New file.
      	* g++.target/arm/pr81497.C: Likewise.
      
      From-SVN: r261191
      Kyrylo Tkachov committed
    • Simplify gcov_histogram as it's used only for ARCS counters. · 7f3577f5
      2018-06-05  Martin Liska  <mliska@suse.cz>
      
      	* auto-profile.c (read_autofdo_file): Do not use
      	gcov_ctr_summary struct.
      	(afdo_callsite_hot_enough_for_early_inline): Likewise.
      	* coverage.c (struct counts_entry): Likewise.
      	(read_counts_file): Read just single summary entry.
      	(get_coverage_counts): Use gcov_summary struct.
      	* coverage.h (get_coverage_counts): Likewise.
      	* gcov-dump.c (dump_working_sets): Likewise.
      	(tag_summary): Dump just single summary.
      	* gcov-io.c (gcov_write_summary): Write just histogram
      	summary.
      	(gcov_read_summary): Read just single summary.
      	(compute_working_sets): Use gcov_summary struct.
      	* gcov-io.h (GCOV_TAG_SUMMARY_LENGTH): Remove usage
      	of GCOV_COUNTERS_SUMMABLE.
      	(GCOV_COUNTERS_SUMMABLE): Remove.
      	(GCOV_FIRST_VALUE_COUNTER): Replace with
      	GCOV_COUNTER_V_INTERVAL.
      	(struct gcov_ctr_summary): Remove.
      	(struct gcov_summary): Directly use fields of former
      	gcov_ctr_summary.
      	(compute_working_sets): Use gcov_summary struct.
      	* gcov.c (read_count_file): Do not use ctrs fields.
      	* lto-cgraph.c (merge_profile_summaries): Use gcov_summary
      	struct.
      	* lto-streamer.h (struct GTY): Make profile_info gcov_summary
      	struct.
      	* profile.c: Likewise.
      	* profile.h: Likewise.
      2018-06-05  Martin Liska  <mliska@suse.cz>
      
      	* libgcov-driver.c (gcov_compute_histogram): Remove usage
      	of gcov_ctr_summary.
      	(compute_summary): Do it just for a single summary.
      	(merge_one_data): Likewise.
      	(merge_summary): Simplify as we read just single summary.
      	(dump_one_gcov): Pass proper argument.
      	* libgcov-util.c (compute_one_gcov): Simplify as we have just
      	single summary.
      	(gcov_info_count_all_cold): Likewise.
      	(calculate_overlap): Likewise.
      
      From-SVN: r261189
      Martin Liska committed
    • Print working directory only in intermediate format (PR gcov-profile/84846). · 632b10e1
      2018-06-05  Martin Liska  <mliska@suse.cz>
      
      	PR gcov-profile/84846
      	* gcov.c (output_lines): Print working directory only
      	in intermediate format.
      
      From-SVN: r261188
      Martin Liska committed
    • * include/std/type_traits: Fix comment typos. · d4b695e4
      From-SVN: r261186
      Jonathan Wakely committed
    • Mark some Filesystem tests as XFAIL for mingw · ce6efef8
      Symlinks are not supported on mingw.
      
      	* testsuite/27_io/filesystem/operations/read_symlink.cc: XFAIL for
      	mingw* targets.
      	* testsuite/27_io/filesystem/operations/symlink_status.cc: Likewise.
      	* testsuite/experimental/filesystem/operations/read_symlink.cc:
      	Likewise.
      
      From-SVN: r261185
      Jonathan Wakely committed
    • S/390: Fix __builtin_tbeginc signature · 51e00a12
      The builtin was accidentally defined to have an integer return value.
      Fixed with the attached patch.
      
      gcc/ChangeLog:
      
      2018-06-05  Andreas Krebbel  <krebbel@linux.ibm.com>
      
      	* config/s390/s390-builtin-types.def: Add void function type.
      	* config/s390/s390-builtins.def: Use the function type for the
      	tbeginc builtin.
      
      gcc/testsuite/ChangeLog:
      
      2018-06-05  Andreas Krebbel  <krebbel@linux.ibm.com>
      
      	* gcc.target/s390/htm-builtins-compile-4.c: New test.
      
      From-SVN: r261183
      Andreas Krebbel committed
    • 2018-06-05 François Dumont <fdumont@gcc.gnu.org> · d31238cf
      	* include/bits/stl_tempbuf.h
      	(_Temporary_buffer(_FwdIte, _FwdIte)): Delete, replaced by...
      	(_Temporary_buffer(_FwdIte, size_type)): ...this, new.
      	* include/ext/memory (temporary_buffer<>(_FwdIte, _FwdIte)): Adapt.
      	* include/bits/stl_algo.h (__stable_partition): Adapt.
      	(__inplace_merge): Adapt.
      	(__stable_sort): Adapt.
      
      From-SVN: r261181
      François Dumont committed
    • Daily bump. · ca6f2936
      From-SVN: r261180
      GCC Administrator committed
  2. 04 Jun, 2018 25 commits
    • RISC-V: Don't clobber retval when __builtin_eh_return called. · fd1e52dc
      	gcc/
      	* config/riscv/riscv-protos.h (riscv_expand_epilogue): Change bool arg
      	to int.
      	* config/riscv/riscv.c (riscv_for_each_saved_reg): New args epilogue
      	and maybe_eh_return.  Change regno to unsigned int.  Use new args to
      	handle EH_RETURN_DATA_REGNO registers properly.
      	(riscv_expand_prologue): Pass new args to riscv_for_each_saved_reg.
      	(riscv_expand_epilogue): Update comment.  Change argument name and
      	type.  Update code to use new name and type.  Pass new args to
      	riscv_for_each_saved_reg.  Only use EH_RETURN_STACKADJ_RTX when
      	EXCEPTION_RETURN.
      	* config/riscv/riscv.md (NORMAL_RETURN): New.
      	(SIBCALL_RETURN, EXCEPTION_RETURN): New.
      	(epilogue, sibcall_epilogue): Update riscv_expand_epilogue arg.
      	(eh_return): Call gen_eh_return_internal and emit barrier.
      	(eh_return_internal): Call riscv_expand_epilogue.
      
      From-SVN: r261176
      Jim Wilson committed
    • gimple-ssa-store-merging.c (struct merged_store_group): Move up bit_insertion… · 7f5a3982
      gimple-ssa-store-merging.c (struct merged_store_group): Move up bit_insertion field and declare can_be_merged_into method.
      
      	* gimple-ssa-store-merging.c (struct merged_store_group): Move up
      	bit_insertion field and declare can_be_merged_into method.
      	(merged_store_group::can_be_merged_into): New method.
      	(imm_store_chain_info::coalesce_immediate): Call it to decide whether
      	consecutive non-overlapping stores can be merged.  Turn MEM_REF stores
      	into BIT_INSERT_EXPR stores if the group contains a non-MEM_REF store.
      
      From-SVN: r261173
      Eric Botcazou committed
    • PR libstdc++/85930 fix misplaced alignment-specifier · d552d819
      	PR libstdc++/85930
      	* include/bits/shared_ptr_base.h [!__cpp_rtti]: Include <typeinfo>
      	unconditionally. Remove redundant declaration.
      	[!__cpp_rtti] (_Sp_make_shared_tag::_S_ti): Fix location of
      	alignment-specifier.
      
      From-SVN: r261172
      Jonathan Wakely committed
    • compiler: fix crashes on cyclic var/type references · f0ebf6e3
          
          This patch fixes type traversal to avoid compiler crashes for test
          cases where a type T includes an expression that refers back to the
          type without actually explicitly mentioning T. Examples include
          
            var x [uintptr(unsafe.Sizeof(&x))]byte
            var a [len(a)]int
          
          The fix involves expanding the set of types that the traversal code
          "remembers" (to avoid cycles) to include array types, and introducing an
          additional guard in Builtin_call_expression::do_is_constant to catch
          cyclic type constructs.
          
          Fixes golang/go#25299
          Fixes golang/go#25679
          Fixes golang/go#25315
          Fixes golang/go#25680
          
          Reviewed-on: https://go-review.googlesource.com/115796
      
      From-SVN: r261168
      Ian Lance Taylor committed
    • Add myself as gcov maintainer. · e68086c4
      2018-06-04  Martin Liska  <mliska@suse.cz>
      
      	* MAINTAINERS: Add myself as gcov maintainer.
      
      From-SVN: r261167
      Martin Liska committed
    • re PR middle-end/85955 (ICE in fold_convert_loc, at fold-const.c:2408) · 1b17b994
      2018-06-04  Richard Biener  <rguenther@suse.de>
      
      	PR tree-optimization/85955
      	* builtins.c (fold_builtin_sincos): Convert pointers to
      	destination to appropriate type before dereferencing.
      
      	* gcc.dg/pr85955.c: New testcase.
      
      From-SVN: r261165
      Richard Biener committed
    • * es.po, fr.po: Update. · 9be91885
      From-SVN: r261159
      Joseph Myers committed
    • rs6000: Fix absif2 · 7638d8b6
      Without this patch absif2 always FAILs.  There is no testcase for
      that, nor do we see it during bootstrap, but it is obvious.
      
      
      	* config/rs6000/rs6000.md (abs<mode>2 for FLOAT128): Handle IFmode.
      
      From-SVN: r261158
      Segher Boessenkool committed
    • Define std::fpos special members as defaulted · 389f7757
      	* include/bits/postypes.h (fpos): Define special members as defaulted.
      
      From-SVN: r261156
      Jonathan Wakely committed
    • PR libstdc++/85930 fix misaligned reference · dbda636d
      	PR libstdc++/85930
      	* include/bits/shared_ptr_base.h (_Sp_make_shared_tag::_S_ti): Align
      	the static variable correctly.
      
      From-SVN: r261155
      Jonathan Wakely committed
    • re PR fortran/85981 (ICE in gfc_trans_string_copy, at fortran/trans-expr.c:6539) · 20ce6ade
      2018-06-04  Steven G. Kargl  <kargl@gcc.gnu.org>
      
      	PR fortran/85981
      	* resolve.c (resolve_allocate_deallocate): Check errmsg is default
      	character kind.
      
      2018-06-04  Steven G. Kargl  <kargl@gcc.gnu.org>
      
      	PR fortran/85981
      	* gfortran.dg/allocate_alloc_opt_14.f90: New test.
      	* gfortran.dg/allocate_alloc_opt_1.f90: Update error string.
      	* gfortran.dg/allocate_stat_2.f90: Ditto.
      	* gfortran.dg/deallocate_alloc_opt_1.f90: Ditto.
      
      From-SVN: r261154
      Steven G. Kargl committed
    • [MAINTAINERS] Update email address · 37cca345
      	* MAINTAINERS: Update my email address.
      
      From-SVN: r261153
      Matthew Fortune committed
    • PR c++/61806 - missed SFINAE with partial specialization. · ae177d35
      	* cp-tree.h (deferring_access_check_sentinel): Add deferring_kind
      	parameter to constructor.
      	* pt.c (instantiate_class_template_1): Enable access checking
      	before call to most_specialized_partial_spec.
      
      From-SVN: r261151
      Jason Merrill committed
    • Fix expand_expr_real_1 handling of BLKmode bitfield references · 8566678b
      The handling of bitfield references in expand_expr_real_1 includes:
      
                  machine_mode ext_mode = mode;
      
                  if (ext_mode == BLKmode
                      && ! (target != 0 && MEM_P (op0)
                            && MEM_P (target)
                            && multiple_p (bitpos, BITS_PER_UNIT)))
                    ext_mode = int_mode_for_size (bitsize, 1).else_blk ();
      
                  if (ext_mode == BLKmode)
                    {
                      [...]
                      gcc_assert (MEM_P (op0)
      
      Here "mode" is the TYPE_MODE of the result, so when mode == BLKmode,
      the target must be a MEM if nonnull, since no other rtl objects can
      have BLKmode.  But there's no guarantee that the source value op0 is also
      BLKmode and thus also a MEM: we can reach the assert for any source if
      the bitsize being extracted is larger than the largest integer mode
      (or larger than MAX_FIXED_MODE_SIZE).
      
      This triggered for SVE with -msve-vector-bits=512, where we could
      sometimes try to extract a BLKmode value from a 512-bit vector,
      and where int_mode_for_size would rightly fail for large bitsizes.
      
      The patch reuses the existing:
      
      	/* Otherwise, if this is a constant or the object is not in memory
      	   and need be, put it there.  */
      	else if (CONSTANT_P (op0) || (!MEM_P (op0) && must_force_mem))
      	  {
      	    memloc = assign_temp (TREE_TYPE (tem), 1, 1);
      	    emit_move_insn (memloc, op0);
      	    op0 = memloc;
      	    clear_mem_expr = true;
      	  }
      
      to handle this case.
      
      2018-05-29  Richard Sandiford  <richard.sandiford@linaro.org>
      
      gcc/
      	* expr.c (expand_expr_real_1): Force the operand into memory if
      	its TYPE_MODE is BLKmode and if there is no integer mode for
      	the number of bits being extracted.
      
      gcc/testsuite/
      	* gcc.target/aarch64/sve/extract_5.c: New test.
      
      From-SVN: r261150
      Richard Sandiford committed
    • re PR target/85832 ([AVX512] possible shorter code when comparing with vector of zeros) · 1fc9aa54
      	PR target/85832
      	PR target/86036
      	* config/i386/sse.md (<avx512>_eq<mode>3<mask_scalar_merge_name>_1):
      	Use vptestnm rather than vptestm in (=Yc,v,C) variant.
      
      	* gcc.target/i386/avx512f-pr85832.c: Expect vptestnm rather than
      	vptestm.
      	* gcc.target/i386/avx512vl-pr85832.c: Likewise.
      	* gcc.target/i386/avx512vlbw-pr85832.c: Likewise.
      	* gcc.target/i386/avx512bw-pr85832.c: Likewise.
      	* gcc.target/i386/avx512bw-pr86036.c: New test.
      
      From-SVN: r261148
      Jakub Jelinek committed
    • PR c++/85765 - SFINAE and non-type default template arg. · 92a38cfd
      	* pt.c (type_unification_real): Do full semantic processing if
      	substituting a partial args list replaces all template parms.
      
      From-SVN: r261146
      Jason Merrill committed
    • tree-cfgcleanup.c (cleanup_tree_cfg_1): Fold into... · 1ccaa21f
      2018-06-04  Richard Biener  <rguenther@suse.de>
      
      	* tree-cfgcleanup.c (cleanup_tree_cfg_1): Fold into...
      	(cleanup_tree_cfg_noloop): ... single caller.  Do
      	start_recording_case_labels later.
      
      From-SVN: r261145
      Richard Biener committed
    • cldemoteintrin.h: Change define from _X86INTRIN_H_INCLUDED to _IMMINTRIN_H_INCLUDED. · 5a6e3d79
      2018-06-04  Sebastian Peryt  <sebastian.peryt@intel.com>
      
              * config/i386/cldemoteintrin.h: Change define from _X86INTRIN_H_INCLUDED
              to _IMMINTRIN_H_INCLUDED.
              * config/i386/pconfigintrin.h: Ditto.
              * config/i386/waitpkgintrin.h: Ditto.
              * config/i386/immintrin.h: Add includes for sgxintrin.h,
              pconfigintrin.h, waitpkgintrin.h and cldemoteintrin.h.
              * config/i386/x86intrin.h: Remove includes for mintrin.h, xmmintrin.h,
              emmintrin.h, pmmintrin.h, tmmintrin.h, smmintrin.h, wmmintrin.h,
              bmiintrin.h, bmi2intrin.h, lzcntintrin.h, sgxintrin.h, pconfigintrin.h,
              waitpkgintrin.h and cldemoteintrin.h.
      
      From-SVN: r261144
      Sebastian Peryt committed
    • [MAINTAINERS] Remove redundant write-after-approval entries · 74386680
      2018-06-04  Tom de Vries  <tdevries@suse.de>
      
      	* MAINTAINERS: Remove write-after-approval entries for component
      	maintainers.
      
      From-SVN: r261143
      Tom de Vries committed
    • re PR tree-optimization/86038 (ICE in to_reg_br_prob_base, at profile-count.h:242) · c77bb674
      2018-06-04  Richard Biener  <rguenther@suse.de>
      
      	PR tree-optimization/86038
      	* tracer.c (find_best_successor): Check probability for
      	being initialized, bail out if not.
      
      	* gcc.dg/pr86038.c: New testcase.
      
      From-SVN: r261142
      Richard Biener committed
    • [arm] PR target/86003 build failures with --with-cpu=xscale · 34a1d5c2
      The XScale cpu configuration in GCC has always been somewhat
      non-conforming.  Although XScale isn't an architecture (it's simply an
      implementation of ARMv5te), we do by tradition emit a specific
      pre-define for it.  We achieve this effect by adding an additional
      feature bit to the xscale CPU definition that isn't part of the base
      architecture.
      
      When I restructured the options last year I overlooked this oddity and
      the result, of course, is that this configuration now fails to build
      as intended.
      
      What happens is that the driver (correctly) constructs an architecture
      for the xscale cpu name (as armv5te) and passes it in addition to the
      CPU name.  The backend code, on finding both a cpu and an architecture
      specifies attempts to correlate the two and finds a difference due to
      the additional feature bit and reports an inconsistency (fatally if
      -werror is specified).
      
      I think the best fix to this is to treat the xscale feature bit using
      the same mechanism that we use for other 'quirks' in CPU
      implementations and simply filter it out before comparing the
      capabilities.  It has the additional benefit that it's also the
      simplest fix.
      
      	PR target/86003
      	* config/arm/arm-cpus.in (ALL_QUIRKS): Add xscale feature to the list
      	of bits to ignore when comparing architectures.
      
      From-SVN: r261140
      Richard Earnshaw committed
    • re PR tree-optimization/69615 (0 to limit signed range checks don't always use unsigned compare) · 261ef15d
      	PR tree-optimization/69615
      	* fold-const.c (merge_ranges): If range1 is - [x, x] and x is the
      	maximum or minimum of the type, try to merge it also as if
      	range1 is + [-, x - 1] or + [x + 1, -].
      
      	* gcc.dg/pr69615.c: New test.
      
      From-SVN: r261139
      Jakub Jelinek committed
    • re PR c++/86025 (ICE with -Wduplicated-branches and OpenMP critical) · 20b11fd4
      	PR c++/86025
      	* tree.c (inchash::add_expr): Handle IDENTIFIER_NODE.
      
      	* c-c++-common/gomp/pr86025.c: New test.
      
      From-SVN: r261137
      Jakub Jelinek committed
    • Daily bump. · ea476799
      From-SVN: r261136
      GCC Administrator committed