1. 01 Aug, 2018 23 commits
    • [03/11] Remove vect_transform_stmt grouped_store argument · b0b45e58
      Nothing now uses the grouped_store value passed back by
      vect_transform_stmt, so we might as well remove it.
      
      2018-08-01  Richard Sandiford  <richard.sandiford@arm.com>
      
      gcc/
      	* tree-vectorizer.h (vect_transform_stmt): Remove grouped_store
      	argument.
      	* tree-vect-stmts.c (vect_transform_stmt): Likewise.
      	* tree-vect-loop.c (vect_transform_loop_stmt): Update call accordingly.
      	(vect_transform_loop): Likewise.
      	* tree-vect-slp.c (vect_schedule_slp_instance): Likewise.
      
      From-SVN: r263216
      Richard Sandiford committed
    • [02/11] Remove vect_schedule_slp return value · 8fe1bd30
      Nothing now uses the vect_schedule_slp return value, so it's not worth
      propagating the value through vect_schedule_slp_instance.
      
      2018-08-01  Richard Sandiford  <richard.sandiford@arm.com>
      
      gcc/
      	* tree-vectorizer.h (vect_schedule_slp): Return void.
      	* tree-vect-slp.c (vect_schedule_slp_instance): Likewise.
      	(vect_schedule_slp): Likewise.
      
      From-SVN: r263215
      Richard Sandiford committed
    • [01/11] Schedule SLP earlier · 99615cf5
      vect_transform_loop used to call vect_schedule_slp lazily when it
      came across the first SLP statement, but it seems easier to do it
      before the main loop.
      
      2018-07-30  Richard Sandiford  <richard.sandiford@arm.com>
      
      gcc/
      	* tree-vect-loop.c (vect_transform_loop_stmt): Remove slp_scheduled
      	argument.
      	(vect_transform_loop): Update calls accordingly.  Schedule SLP
      	instances before the main loop, if any exist.
      
      From-SVN: r263214
      Richard Sandiford committed
    • Fix over-widening handling of COND_EXPRs (PR 86749) · 047fba34
      This PR is a wrong-code bug caused by the over-widening support.
      The minimum input precisions for a COND_EXPR are supposed to apply
      only to the "then" and "else" values, but here we were applying
      them to the operands of a nested COND_EXPR comparison instead.
      
      2018-08-01  Richard Sandiford  <richard.sandiford@arm.com>
      
      gcc/
      	PR tree-optimization/86749
      	* tree-vect-patterns.c (vect_determine_min_output_precision_1):
      	If the lhs is used in a COND_EXPR, check that it is being used
      	as the "then" or "else" value.
      
      gcc/testsuite/
      	PR tree-optimization/86749
      	* gcc.dg/vect/pr86749.c: New test.
      
      From-SVN: r263213
      Richard Sandiford committed
    • [PATCH] Remove use of 'struct map' from plugin (nvptx) · 094db6be
      	libgomp/
      	* plugin/plugin-nvptx.c (struct map): Removed.
      	(map_init, map_pop): Remove use of struct map. (map_push):
      	Likewise and change argument list.
      	* testsuite/libgomp.oacc-c-c++-common/mapping-1.c: New
      
      Co-Authored-By: James Norris <jnorris@codesourcery.com>
      
      From-SVN: r263212
      Cesar Philippidis committed
    • PR libstdc++/60555 std::system_category() should recognise POSIX errno values · 5ecfbf82
      	PR libstdc++/60555
      	* src/c++11/system_error.cc
      	(system_error_category::default_error_condition): New override to
      	check for POSIX errno values.
      	* testsuite/19_diagnostics/error_category/generic_category.cc: New
      	* testsuite/19_diagnostics/error_category/system_category.cc: New
      	test.
      
      From-SVN: r263210
      Jonathan Wakely committed
    • [nvptx] Define TARGET_HAVE_SPECULATION_SAFE_VALUE · e335138d
      2018-08-01  Tom de Vries  <tdevries@suse.de>
      
      	PR target/86800
      	* config/nvptx/nvptx.c (TARGET_HAVE_SPECULATION_SAFE_VALUE): Define to
      	speculation_safe_value_not_needed.
      
      From-SVN: r263209
      Tom de Vries committed
    • [libgomp, nvptx] Add cuda-lib.def · 8c6310a2
      2018-08-01  Tom de Vries  <tdevries@suse.de>
      
      	* plugin/cuda-lib.def: New file.  Factor out of ...
      	* plugin/plugin-nvptx.c (CUDA_CALLS): ... here.
      	(struct cuda_lib_s, init_cuda_lib): Include cuda-lib.def instead of
      	using CUDA_CALLS.
      
      From-SVN: r263208
      Tom de Vries committed
    • re PR c++/86661 (g++ ICE:tree check: expected tree that contains ‘decl minimal’… · 5ebbb72c
      re PR c++/86661 (g++ ICE:tree check: expected tree that contains ‘decl minimal’ structure, have ‘overload’ in note_name_declared_in_class, at cp/class.c:8288)
      
      /cp
      2018-08-01  Paolo Carlini  <paolo.carlini@oracle.com>
      
      	PR c++/86661
      	* class.c (note_name_declared_in_class): Use location_of in permerror
      	instead of DECL_SOURCE_LOCATION (for OVERLOADs).
      
      /testsuite
      2018-08-01  Paolo Carlini  <paolo.carlini@oracle.com>
      
      	PR c++/86661
      	* g++.dg/lookup/name-clash12.C: New.
      
      From-SVN: r263207
      Paolo Carlini committed
    • tree-ssa-sccvn.c (visit_phi): Compare invariant addresses as base and offset. · e4837aa9
      2018-08-01  Richard Biener  <rguenther@suse.de>
      
      	* tree-ssa-sccvn.c (visit_phi): Compare invariant addresses
      	as base and offset.
      
      	* gcc.dg/tree-ssa/ssa-fre-68.c: New testcase.
      
      From-SVN: r263206
      Richard Biener committed
    • poly-int-07_plugin.c (dg-options): Use -O0. · 42c4ccce
      	* gcc.dg/plugin/poly-int-07_plugin.c (dg-options): Use -O0.
      
      From-SVN: r263205
      Uros Bizjak committed
    • pr84512.c: Xfail on alpha*-*-*. · 4bbea044
      	* gcc.dg/tree-ssa/pr84512.c: Xfail on alpha*-*-*.
      
      From-SVN: r263204
      Uros Bizjak committed
    • Improve dumping of value profiling transformations. · 7f87c8da
      2018-08-01  Martin Liska  <mliska@suse.cz>
      
      	* value-prof.c (gimple_divmod_fixed_value_transform): Unify
              format how successful transformation is dumped.
      	(gimple_mod_pow2_value_transform): Likewise.
      	(gimple_mod_subtract_transform): Likewise.
      	(gimple_stringops_transform): Likewise.
      2018-08-01  Martin Liska  <mliska@suse.cz>
      
      	* gcc.dg/tree-prof/stringop-1.c: Adjust scanned pattern.
      	* gcc.dg/tree-prof/stringop-2.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-1.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-2.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-3.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-4.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-5.c: Likewise.
      	* gcc.dg/tree-prof/val-prof-7.c: Likewise.
      
      From-SVN: r263203
      Martin Liska committed
    • __gcov_indirect_call_callee can't be null in __gcov_indirect_call_profiler_v2. · fd2e1dcd
      2018-08-01  Martin Liska  <mliska@suse.cz>
      
      	* libgcov-profiler.c (__gcov_indirect_call_profiler_v2): Do not
              check that  __gcov_indirect_call_callee is non-null.
      
      From-SVN: r263202
      Martin Liska committed
    • Add memmove to value profiling. · 181f2e99
      2018-08-01  Martin Liska  <mliska@suse.cz>
      
              PR value-prof/35543
      	* value-prof.c (interesting_stringop_to_profile_p):
              Simplify the code and add BUILT_IN_MEMMOVE.
      	(gimple_stringops_transform): Likewise.
      2018-08-01  Martin Liska  <mliska@suse.cz>
      
              PR value-prof/35543
      	* gcc.dg/tree-prof/val-prof-7.c: Add __builtin_memmove.
      
      From-SVN: r263201
      Martin Liska committed
    • [PATCH][AArch64] Stop redundant zero-extension after UMOV when in DI mode · 00d29b97
      This patch extends the aarch64_get_lane_zero_extendsi instruction
      definition to also cover DI mode. This prevents a redundant AND
      instruction from being generated due to the pattern failing to be matched.
      
      Committed on behalf of Sam Tebbs.
      
      gcc/
      2018-08-01  Sam Tebbs  <sam.tebbs@arm.com>
      
      	* config/aarch64/aarch64-simd.md
      	(*aarch64_get_lane_zero_extendsi<mode>): Rename to...
      	(*aarch64_get_lane_zero_extend<GPI:mode><VDQQH:mode>): ... This and
      	use GPI iterator instead of SI mode.
      
      gcc/testsuite
      2018-08-01  Sam Tebbs  <sam.tebbs@arm.com>
      
      	* gcc.target/aarch64/extract_zero_extend.c: New file.
      
      From-SVN: r263200
      Sam Tebbs committed
    • re PR c/85704 (cc1 run out of memory when it compile) · 5922dcb5
      	PR c/85704
      	* c-typeck.c (init_field_decl_cmp): New function.
      	(output_pending_init_elements): Use it for field comparisons
      	instead of pure bit_position comparisons.
      
      	* gcc.c-torture/compile/pr85704.c: New test.
      
      From-SVN: r263198
      Jakub Jelinek committed
    • rs6000 - add speculation_barrier pattern · d1b05d12
      This patch reworks the existing rs6000_speculation_barrier pattern to
      work with the new __builtin_sepculation_safe_value() intrinsic.  The
      change is trivial as it simply requires renaming the existing speculation
      barrier pattern.
      
      So the total patch is to delete 14 characters!
      
      	* config/rs6000/rs6000.md (speculation_barrier): Renamed from
      	rs6000_speculation_barrier.
      	* config/rs6000/rs6000.c (rs6000_expand_builtin): Adjust for
      	new barrier pattern name.
      
      From-SVN: r263197
      Richard Earnshaw committed
    • x86 - add speculation_barrier pattern · 31e962f2
      This patch adds a speculation barrier for x86, based on my
      understanding of the required mitigation for that CPU, which is to use
      an lfence instruction.
      
      This patch needs some review by an x86 expert and if adjustments are
      needed, I'd appreciate it if they could be picked up by the port
      maintainer.  This is supposed to serve as an example of how to deploy
      the new __builtin_speculation_safe_value() intrinsic on this
      architecture.
      
      	* config/i386/i386.md (unspecv): Add UNSPECV_SPECULATION_BARRIER.
      	(speculation_barrier): New insn.
      
      From-SVN: r263196
      Richard Earnshaw committed
    • re PR tree-optimization/86724 (Compilation error with new isl 0.20 (missing includes)) · 71f10c42
      2018-08-01  Richard Biener  <rguenther@suse.de>
      
      	PR bootstrap/86724
      	* graphite.h: Include isl/id.h and isl/space.h to allow build
      	with ISL 0.20.
      
      From-SVN: r263193
      Richard Biener committed
    • re PR target/86651 (lto-wrapper.exe: fatal error:… · 3311d93c
      re PR target/86651 (lto-wrapper.exe: fatal error: simple_object_copy_lto_debug_sections not implemented: Invalid argument)
      
      2018-08-01 Jan Willem Jagersma  <jwjagersma@gmail.com>
      
      	PR target/86651
      	* dwarf2out.c (dwarf2out_early_finish): Do not generate assembly in LTO
      	mode for COFF targets.
      	* defaults.h (TARGET_COFF): Define.
      	* config/i386/djgpp.h (TARGET_ASM_LTO_START, TARGET_ASM_LTO_END,
      	TARGET_COFF): Define.
      	(i386_djgpp_asm_lto_start, i386_djgpp_asm_lto_end): Declare.
      	* config/i386/djgpp.c (saved_debug_info_level): New static variable.
      	(i386_djgpp_asm_lto_start, i386_djgpp_asm_lto_end): New functions.
      
      From-SVN: r263191
      Jan Willem Jagersma committed
    • Daily bump. · 027c7141
      From-SVN: r263190
      GCC Administrator committed
    • runtime: use poll rather than pollset for netpoll on AIX · 67e77214
          
          Updates golang/go#26634
          
          Reviewed-on: https://go-review.googlesource.com/126857
      
      From-SVN: r263186
      Ian Lance Taylor committed
  2. 31 Jul, 2018 17 commits
    • PR libstdc++/86751 default assignment operators for std::pair · e1821582
      The solution for PR 77537 causes ambiguities due to the extra copy
      assignment operator taking a __nonesuch_no_braces parameter. By making
      the base class non-assignable we don't need the extra deleted overload
      in std::pair. The copy assignment operator will be implicitly deleted
      (and the move assignment operator not declared) as needed. Without the
      additional user-provided operator in std::pair the ambiguity is avoided.
      
      	PR libstdc++/86751
      	* include/bits/stl_pair.h (__pair_base): New class with deleted copy
      	assignment operator.
      	(pair): Derive from __pair_base.
      	(pair::operator=): Remove deleted overload.
      	* python/libstdcxx/v6/printers.py (StdPairPrinter): New pretty printer
      	so that new base class isn't shown in GDB.
      	* testsuite/20_util/pair/86751.cc: New test.
      	* testsuite/20_util/pair/ref_assign.cc: New test.
      
      From-SVN: r263185
      Jonathan Wakely committed
    • Don't unconditionally define feature test macros in <version> · ff273400
      The macro definitions in <version> should depend on the same
      preprocessor conditions as the original macros in other headers.
      Otherwise <version> can define macros that imply the availability of
      features that are not actually defined.
      
      This fix is incomplete, as __cpp_lib_filesystem should depend on whether
      libstdc++fs.a is supported, and several macros should only be defined
      when _GLIBCXX_HOSTED is defined. Also, the feature test macros should
      define their value as type long, but most are type int.
      
      	* include/bits/c++config (_GLIBCXX_HAVE_BUILTIN_HAS_UNIQ_OBJ_REP)
      	(_GLIBCXX_HAVE_BUILTIN_IS_AGGREGATE): Move definitions here.
      	(_GLIBCXX_HAVE_BUILTIN_LAUNDER): Likewise. Use !__is_identifier
      	instead of __has_builtin.
      	* include/std/type_traits (_GLIBCXX_HAVE_BUILTIN_HAS_UNIQ_OBJ_REP)
      	(_GLIBCXX_HAVE_BUILTIN_IS_AGGREGATE): Remove definitions from here.
      	* include/std/version [!_GLIBCXX_HAS_GTHREADS]
      	(__cpp_lib_shared_timed_mutex, __cpp_lib_scoped_lock)
      	(__cpp_lib_shared_mutex): Don't define when Gthreads not in use.
      	[!_GLIBCXX_HAVE_BUILTIN_HAS_UNIQ_OBJ_REP]
      	(__cpp_lib_has_unique_object_representations): Don't define when
      	builtin not available.
      	[!_GLIBCXX_HAVE_BUILTIN_IS_AGGREGATE] (__cpp_lib_is_aggregate):
      	Likewise.
      	[!_GLIBCXX_HAVE_BUILTIN_LAUNDER] (__cpp_lib_launder): Likewise.
      	* libsupc++/new (_GLIBCXX_HAVE_BUILTIN_LAUNDER): Remove definition
      	from here.
      
      From-SVN: r263184
      Jonathan Wakely committed
    • Save discriminator info for LTO · b348c78a
      for  gcc/ChangeLog
      
      	* gimple-streamer-in.c (input_bb): Restore BB discriminator.
      	* gimple-streamer-out.c (output_bb): Save it.
      	* lto-streamer-in.c (input_struct_function_base): Restore
      	instance discriminator if available.  Create map on demand.
      	* lto-streamer-out.c (output_struct_function_base): Save it if
      	available.
      	* final.c (decl_to_instance_map): Document LTO strategy.
      
      From-SVN: r263183
      Alexandre Oliva committed
    • Introduce instance discriminators · fa6fd7b7
      With -gnateS, the Ada compiler sets itself up to output discriminators
      for different instantiations of generics, but the middle and back ends
      have lacked support for that.  This patch introduces the missing bits,
      translating the GNAT-internal representation of the per-file instance
      map to an instance_table that maps decls to instance discriminators.
      
      
      From: Alexandre Oliva  <oliva@adacore.com>, Olivier Hainque  <hainque@adacore.com>
      for  gcc/ChangeLog
      
      	* debug.h (decl_to_instance_map_t): New type.
      	(decl_to_instance_map): Declare.
      	(maybe_create_decl_to_instance_map): New inline function.
          	* final.c (bb_discriminator, last_bb_discriminator): New statics,
          	to track basic block discriminators.
          	(final_start_function_1): Initialize them.
          	(final_scan_insn_1): On NOTE_INSN_BASIC_BLOCK, track
      	bb_discriminator.
      	(decl_to_instance_map): New variable.
      	(map_decl_to_instance, maybe_set_discriminator): New functions.
          	(notice_source_line): Set discriminator.
      
      for  gcc/ada/ChangeLog
      
      	* trans.c: Include debug.h.
      	(file_map): New static variable.
      	(gigi): Set it.  Create decl_to_instance_map when needed.
      	(Subprogram_Body_to_gnu): Pass gnu_subprog_decl to...
      	(Sloc_to_locus): ... this.  Add decl parm, map it to instance.
      	* gigi.h (Sloc_to_locus): Adjust declaration.
      
      for  gcc/testsuite/ChangeLog
      
      	* gnat.dg/dinst.adb: New.
      	* gnat.dg/dinst_pkg.ads, gnat.dg/dinst_pkg.adb: New.
      
      Co-Authored-By: Olivier Hainque <hainque@adacore.com>
      
      From-SVN: r263182
      Alexandre Oliva committed
    • c-family: clean up the data tables in c-format.c · e540ccc0
      The format_char_info tables in c-format.c for our own formats contain
      a lot of repetition.
      
      This patch adds a macro to express the conversion specifiers implemented
      within pp_format, making it clearer which are custom ones added by the
      various diagnostic_format_decoder callbacks.
      
      Doing so uncovered a few mistakes in the data (based on comparison with
      the source of the diagnostic_format_decoder callbacks, and the notes
      below), which the patch fixes:
      
      - gcc_diag_char_table didn't have 'Z', but it *is* implemented by pp_format.
      
      - removed erroneous 'G' and 'K' entries from gcc_diag_char_table: they're
        implemented by default_tree_printer (and thus in "tdiag") and by the
        C/C++ FEs, but not in pp_format.
      
      - removed "v" (lower case) from gcc_tdiag_char_table and
        gcc_cxxdiag_char_table
      
      Notes:
      
      pretty-print.h uses this for ATTRIBUTE_GCC_PPDIAG, used by pp_printf
      and pp_verbatim:
      
      whereas diagnostic-core.h uses this for ATTRIBUTE_GCC_DIAG, used by
      the various diagnostic functions:
      
      /* If we haven't already defined a front-end-specific diagnostics
         style, use the generic one.  */
      
      Hence I'm assuming that __gcc_diag__ is for use for when we don't
      know what kind of diagnostic_format_decoder we have, and we can
      only rely on pp_format's core functionality, where __gcc_tdiag__
      is allowed to assume default_tree_printer.
      
      gcc/c-family/ChangeLog:
      	* c-format.c (PP_FORMAT_CHAR_TABLE): New macro, based on existing
      	table entries for gcc_diag_char_table, and the 'Z' entry from
      	gcc_tdiag_char_table, changing the "chain" entry for 'Z' from
      	&gcc_tdiag_char_table[0] to &gcc_diag_char_table[0].
      	(gcc_diag_char_table): Use PP_FORMAT_CHAR_TABLE, implicitly
      	adding missing "Z" for this table.  Remove erroneous "G" and "K"
      	entries.
      	(gcc_tdiag_char_table): Use PP_FORMAT_CHAR_TABLE.  Remove "v".
      	(gcc_cdiag_char_table): Use PP_FORMAT_CHAR_TABLE.
      	(gcc_cxxdiag_char_table): Use PP_FORMAT_CHAR_TABLE.  Remove "v".
      
      gcc/testsuite/ChangeLog:
      	* gcc.dg/format/gcc_diag-1.c (foo): Update the %v tests for
      	tdiag and cxxdiag.
      	* gcc.dg/format/gcc_diag-10.c (test_diag): Update tests of %G
      	and %K.
      
      From-SVN: r263181
      David Malcolm committed
    • targhooks.c (default_have_speculation_safe_value): Add ATTRIBUTE_UNUSED. · 31d87dd2
      	* targhooks.c (default_have_speculation_safe_value): Add
      	ATTRIBUTE_UNUSED.
      
      From-SVN: r263180
      Ian Lance Taylor committed
    • dumpfile.c: eliminate special-casing of dump_file/alt_dump_file · b84e3bde
      With the addition of optinfo, the various dump_* calls had three parts:
      - optionally print to dump_file
      - optionally print to alt_dump_file
      - optionally make an optinfo_item and add it to the pending optinfo,
        creating it for dump_*_loc calls.
      
      However, this split makes it difficult to implement the formatted dumps
      later in patch kit, so as enabling work towards that, this patch removes
      the above split, so that all dumping within the dump_* API goes through
      optinfo_item.
      
      In order to ensure that the dumps to dump_file and alt_dump_file are
      processed immediately (rather than being buffered within the pending
      optinfo for consolidation), this patch introduces the idea of "immediate"
      optinfo_item destinations vs "non-immediate" destinations.
      
      The patch also adds selftest coverage of what's printed, and of scopes.
      
      This adds two allocations per dump_* call when dumping is enabled.
      I'm assuming that this isn't a problem, as dump_enabled_p is normally
      false.  There are ways of optimizing it if it is an issue (by making
      optinfo_item instances become temporaries that borrow the underlying
      buffer), but they require nontrivial changes, so I'd prefer to leave
      that for another patch kit, if it becomes necessary.
      
      gcc/ChangeLog:
      	* dump-context.h: Include "pretty-print.h".
      	(dump_context::refresh_dumps_are_enabled): New decl.
      	(dump_context::emit_item): New decl.
      	(class dump_context): Add fields "m_test_pp" and
      	"m_test_pp_flags".
      	(temp_dump_context::temp_dump_context): Add param "test_pp_flags".
      	(temp_dump_context::get_dumped_text): New decl.
      	(class temp_dump_context): Add field "m_pp".
      	* dumpfile.c (refresh_dumps_are_enabled): Convert to...
      	(dump_context::refresh_dumps_are_enabled): ...and add a test for
      	m_test_pp.
      	(set_dump_file): Update for above change.
      	(set_alt_dump_file): Likewise.
      	(dump_loc): New overload, taking a pretty_printer *.
      	(dump_context::dump_loc): Call end_any_optinfo.  Dump the location
      	to any test pretty-printer.
      	(make_item_for_dump_gimple_stmt): New function, adapted from
      	optinfo::add_gimple_stmt.
      	(dump_context::dump_gimple_stmt): Call it, and use the result,
      	eliminating the direct usage of dump_file and alt_dump_file in
      	favor of indirectly using them via emit_item.
      	(make_item_for_dump_gimple_expr): New function, adapted from
      	optinfo::add_gimple_expr.
      	(dump_context::dump_gimple_expr): Call it, and use the result,
      	eliminating the direct usage of dump_file and alt_dump_file in
      	favor of indirectly using them via emit_item.
      	(make_item_for_dump_generic_expr): New function, adapted from
      	optinfo::add_tree.
      	(dump_context::dump_generic_expr): Call it, and use the result,
      	eliminating the direct usage of dump_file and alt_dump_file in
      	favor of indirectly using them via emit_item.
      	(make_item_for_dump_printf_va): New function, adapted from
      	optinfo::add_printf_va.
      	(make_item_for_dump_printf): New function.
      	(dump_context::dump_printf_va): Call make_item_for_dump_printf_va,
      	and use the result, eliminating the direct usage of dump_file and
      	alt_dump_file in favor of indirectly using them via emit_item.
      	(make_item_for_dump_dec): New function.
      	(dump_context::dump_dec): Call it, and use the result,
      	eliminating the direct usage of dump_file and alt_dump_file in
      	favor of indirectly using them via emit_item.
      	(make_item_for_dump_symtab_node): New function, adapted from
      	optinfo::add_symtab_node.
      	(dump_context::dump_symtab_node): Call it, and use the result,
      	eliminating the direct usage of dump_file and alt_dump_file in
      	favor of indirectly using them via emit_item.
      	(dump_context::begin_scope): Reimplement, avoiding direct usage
      	of dump_file and alt_dump_file in favor of indirectly using them
      	via emit_item.
      	(dump_context::emit_item): New member function.
      	(temp_dump_context::temp_dump_context): Add param "test_pp_flags".
      	Set up test pretty-printer on the underlying context.  Call
      	refresh_dumps_are_enabled.
      	(temp_dump_context::~temp_dump_context): Call
      	refresh_dumps_are_enabled.
      	(temp_dump_context::get_dumped_text): New member function.
      	(selftest::verify_dumped_text): New function.
      	(ASSERT_DUMPED_TEXT_EQ): New macro.
      	(selftest::test_capture_of_dump_calls): Run all tests twice, with
      	and then without optinfo enabled.  Add uses of
      	ASSERT_DUMPED_TEXT_EQ to all tests.  Add test of nested scopes.
      	* dumpfile.h: Update comment for the dump_* API.
      	* optinfo-emit-json.cc
      	(selftest::test_building_json_from_dump_calls): Update for new
      	param for temp_dump_context ctor.
      	* optinfo.cc (optinfo_item::optinfo_item): Remove "owned" param
      	and "m_owned" field.
      	(optinfo_item::~optinfo_item): Likewise.
      	(optinfo::add_item): New member function.
      	(optinfo::emit): Update comment.
      	(optinfo::add_string): Delete.
      	(optinfo::add_printf): Delete.
      	(optinfo::add_printf_va): Delete.
      	(optinfo::add_gimple_stmt): Delete.
      	(optinfo::add_gimple_expr): Delete.
      	(optinfo::add_tree): Delete.
      	(optinfo::add_symtab_node): Delete.
      	(optinfo::add_dec): Delete.
      	* optinfo.h (class dump_context): New forward decl.
      	(optinfo::add_item): New decl.
      	(optinfo::add_string): Delete.
      	(optinfo::add_printf): Delete.
      	(optinfo::add_printf_va): Delete.
      	(optinfo::add_gimple_stmt): Delete.
      	(optinfo::add_gimple_expr): Delete.
      	(optinfo::add_tree): Delete.
      	(optinfo::add_symtab_node): Delete.
      	(optinfo::add_dec): Delete.
      	(optinfo::add_poly_int): Delete.
      	(optinfo_item::optinfo_item): Remove "owned" param.
      	(class optinfo_item): Remove field "m_owned".
      
      From-SVN: r263178
      David Malcolm committed
    • re PR middle-end/86705 (pr45678-2.c ICE with msp430-elf -mlarge) · b06e1dce
      	PR middle-end/86705
      
      	* gcc/cfgexpand.c (set_parm_rtl): Use the alignment of Pmode when
      	MAX_SUPPORTED_STACK_ALIGNMENT would otherwise be exceeded by the
      	requested variable alignment.
      	(expand_one_ssa_partition): Likewise.
      	(expand_one_var): Likewise.
      
      From-SVN: r263177
      Jozef Lawrynowicz committed
    • pdp11 - example of a port not needing a speculation barrier · 31f52518
      This patch is intended as an example of all that is needed if the
      target system doesn't support CPUs that have speculative execution.
      I've chosen the pdp11 port on the basis that it's old enough that this
      is likely to be true for all existing implementations and that there
      is also little chance of that changing in future!
      
      	* config/pdp11/pdp11.c (TARGET_HAVE_SPECULATION_SAFE_VALUE): Redefine
      	to speculation_safe_value_not_needed.
      
      From-SVN: r263176
      Richard Earnshaw committed
    • targhooks - provide an alternative hook for targets that never execute speculatively · f3debef3
      This hook adds an alternative implementation for the target hook
      TARGET_HAVE_SPECULATION_SAFE_VALUE; it can be used by targets that have no
      CPU implementations that execute code speculatively.  All that is needed for
      such targets now is to add:
      
       #undef TARGET_HAVE_SPECULATION_SAFE_VALUE
       #define TARGET_HAVE_SPECULATION_SAFE_VALUE speculation_safe_value_not_needed.
      
      to where you have your other target hooks and you're done.
      
      gcc:
      	* targhooks.h (speculation_safe_value_not_needed): New prototype.
      	* targhooks.c (speculation_safe_value_not_needed): New function.
      	* target.def (have_speculation_safe_value): Update documentation.
      	* doc/tm.texi: Regenerated.
      
      From-SVN: r263175
      Richard Earnshaw committed
    • AArch64 - use CSDB based sequences if speculation tracking is enabled · c0111dc4
      In this final patch, now that we can track speculation through conditional
      branches, we can use this information to use a less expensive CSDB based
      speculation barrier.
      
      	* config/aarch64/iterators.md (ALLI_TI): New iterator.
      	* config/aarch64/aarch64.md (despeculate_copy<ALLI_TI:mode>): New
      	expand.
      	(despeculate_copy<ALLI:mode>_insn): New insn.
      	(despeculate_copyti_insn): New insn.
      	(despeculate_simple<ALLI:mode>): New insn
      	(despeculate_simpleti): New insn.
      	* config/aarch64/aarch64.c (aarch64_speculation_safe_value): New
      	function.
      	(TARGET_SPECULATION_SAFE_VALUE): Redefine to
      	aarch64_speculation_safe_value.
      	(aarch64_print_operand): Handle const0_rtx in modifier 'H'.
      
      From-SVN: r263174
      Richard Earnshaw committed
    • AArch64 - new pass to add conditional-branch speculation tracking · 3751345d
      This patch is the main part of the speculation tracking code.  It adds
      a new target-specific pass that is run just before the final branch
      reorg pass (so that it can clean up any new edge insertions we make).
      The pass is only run with -mtrack-speculation is passed on the command
      line.
      
      One thing that did come to light as part of this was that the stack pointer
      register was not being permitted in comparision instructions.  We rely on
      that for moving the tracking state between SP and the scratch register at
      function call boundaries.
      
      	* config/aarch64/aarch64-speculation.cc: New file.
      	* config/aarch64/aarch64-passes.def (pass_track_speculation): Add before
      	pass_reorder_blocks.
      	* config/aarch64/aarch64-protos.h (make_pass_track_speculation): Add
      	prototype.
      	* config/aarch64/aarch64.c (aarch64_conditional_register_usage): Fix
      	X14 and X15 when tracking speculation.
      	* config/aarch64/aarch64.md (register name constants): Add
      	SPECULATION_TRACKER_REGNUM and SPECULATION_SCRATCH_REGNUM.
      	(unspec): Add UNSPEC_SPECULATION_TRACKER.
      	(speculation_barrier): New insn attribute.
      	(cmp<mode>): Allow SP in comparisons.
      	(speculation_tracker): New insn.
      	(speculation_barrier): Add speculation_barrier attribute.
      	* config/aarch64/t-aarch64: Add make rule for aarch64-speculation.o.
      	* config.gcc (aarch64*-*-*): Add aarch64-speculation.o to extra_objs.
      	* doc/invoke.texi (AArch64 Options): Document -mtrack-speculation.
      
      From-SVN: r263173
      Richard Earnshaw committed
    • AArch64 - disable CB[N]Z TB[N]Z when tracking speculation · 6e1eaca9
      The CB[N]Z and TB[N]Z instructions do not expose the comparison through
      the condition code flags.  This makes it impossible to track speculative
      execution through such a branch.  We can handle this relatively easily
      by simply disabling the patterns in this case.
      
      A side effect of this is that the split patterns for the atomic operations
      need to also avoid generating these instructions.  They mostly have simple
      fall-backs for this already.
      
      	* config/aarch64/aarch64.md (cb<optab><mode>1): Disable when
      	aarch64_track_speculation is true.
      	(tb<optab><mode>1): Likewise.
      	* config/aarch64/aarch64.c (aarch64_split_compare_regs): Do not
      	generate CB[N]Z when tracking speculation.
      	(aarch64_split_compare_and_swap): Likewise.
      	(aarch64_split_atomic_op): Likewise.
      
      From-SVN: r263172
      Richard Earnshaw committed
    • AArch64 - Add new option -mtrack-speculation · 3b0c2502
      This patch doesn't do anything useful, it simply adds a new command-line
      option -mtrack-speculation to AArch64.  Subsequent patches build on this.
      
      	* config/aarch64/aarch64.opt (mtrack-speculation): New target option.
      
      From-SVN: r263171
      Richard Earnshaw committed
    • AArch64 - add speculation barrier · def745b2
      Similar to Arm, this adds an unconditional speculation barrier for AArch64.
      
      	* config/aarch64.md (unspecv): Add UNSPECV_SPECULAION_BARRIER.
      	(speculation_barrier): New insn.
      
      From-SVN: r263170
      Richard Earnshaw committed
    • Arm - add speculation_barrier pattern · bb8b0096
      This patch defines a speculation barrier for AArch32.
      
      	* config/arm/unspecs.md (unspecv): Add VUNSPEC_SPECULATION_BARRIER.
      	* config/arm/arm.md (speculation_barrier): New expand.
      	(speculation_barrier_insn): New pattern.
      
      From-SVN: r263169
      Richard Earnshaw committed
    • Add __builtin_speculation_safe_value · 425fc685
      This patch defines a new intrinsic function
      __builtin_speculation_safe_value.  A generic default implementation is
      defined which will attempt to use the backend pattern
      "speculation_safe_barrier".  If this pattern is not defined, or if it
      is not available, then the compiler will emit a warning, but
      compilation will continue.
      
      Note that the test spec-barrier-1.c will currently fail on all
      targets.  This is deliberate, the failure will go away when
      appropriate action is taken for each target backend.
      
      gcc:
      	* builtin-types.def (BT_FN_PTR_PTR_VAR): New function type.
      	(BT_FN_I1_I1_VAR, BT_FN_I2_I2_VAR, BT_FN_I4_I4_VAR): Likewise.
      	(BT_FN_I8_I8_VAR, BT_FN_I16_I16_VAR): Likewise.
      	* builtin-attrs.def (ATTR_NOVOPS_NOTHROW_LEAF_LIST): New attribute
      	list.
      	* builtins.def (BUILT_IN_SPECULATION_SAFE_VALUE_N): New builtin.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_PTR): New internal builtin.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_1): Likewise.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_2): Likewise.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_4): Likewise.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_8): Likewise.
      	(BUILT_IN_SPECULATION_SAFE_VALUE_16): Likewise.
      	* builtins.c (expand_speculation_safe_value): New function.
      	(expand_builtin): Call it.
      	* doc/cpp.texi: Document predefine __HAVE_SPECULATION_SAFE_VALUE.
      	* doc/extend.texi: Document __builtin_speculation_safe_value.
      	* doc/md.texi: Document "speculation_barrier" pattern.
      	* doc/tm.texi.in: Pull in TARGET_SPECULATION_SAFE_VALUE and
      	TARGET_HAVE_SPECULATION_SAFE_VALUE.
      	* doc/tm.texi: Regenerated.
      	* target.def (have_speculation_safe_value, speculation_safe_value): New
      	hooks.
      	* targhooks.c (default_have_speculation_safe_value): New function.
      	(default_speculation_safe_value): New function.
      	* targhooks.h (default_have_speculation_safe_value): Add prototype.
      	(default_speculation_safe_value): Add prototype.
      
      c-family:
      	* c-common.c (speculation_safe_resolve_call): New function.
      	(speculation_safe_resolve_params): New function.
      	(speculation_safe_resolve_return): New function.
      	(resolve_overloaded_builtin): Handle __builtin_speculation_safe_value.
      	* c-cppbuiltin.c (c_cpp_builtins): Add pre-define for
      	__HAVE_SPECULATION_SAFE_VALUE.
      
      testsuite:
      	* c-c++-common/spec-barrier-1.c: New test.
      	* c-c++-common/spec-barrier-2.c: New test.
      	* gcc.dg/spec-barrier-3.c: New test.
      
      From-SVN: r263168
      Richard Earnshaw committed