platform/upstream/gcc.git
8 years agocfg.c (free_original_copy_tables): Replace second assignment of bb_copy = NULL by...
Prathamesh Kulkarni [Fri, 2 Sep 2016 17:00:38 +0000 (17:00 +0000)]
cfg.c (free_original_copy_tables): Replace second assignment of bb_copy = NULL by bb_original = NULL.

2016-09-02  Prathamesh Kulkarni  <prathamesh.kulkarni@linaro.org>

* cfg.c (free_original_copy_tables): Replace second assignment of
bb_copy = NULL by bb_original = NULL.

From-SVN: r239960

8 years agore PR other/77421 (Bugs found in GCC with the help of PVS-Studio)
Jakub Jelinek [Fri, 2 Sep 2016 16:18:35 +0000 (18:18 +0200)]
re PR other/77421 (Bugs found in GCC with the help of PVS-Studio)

PR other/77421
* config/i386/i386.c (ix86_expanded_args_builtin): Remove redundant
assignment added in r216794.

From-SVN: r239959

8 years agoImprovements to typed_splay_tree
David Malcolm [Fri, 2 Sep 2016 15:22:21 +0000 (15:22 +0000)]
Improvements to typed_splay_tree

This patch adds foreach, max and min methods to
class typed_splay_tree, along with the start of a selftest
suite.

gcc/ChangeLog:
* Makefile.in (OBJS): Add typed-splay-tree.o.
* selftest-run-tests.c (selftest::run_tests): Call
typed_splay_tree_c_tests.
* selftest.h (typed_splay_tree_c_tests): New decl.
* typed-splay-tree.c: New file.
* typed-splay-tree.h (typed_splay_tree::foreach_fn): New typedef.
(typed_splay_tree::max): New method.
(typed_splay_tree::min): New method.
(typed_splay_tree::foreach): New method.
(typed_splay_tree::closure): New struct.
(typed_splay_tree::inner_foreach_fn): New function.

From-SVN: r239958

8 years agoUse setrlimit for testing libstdc++ in cross toolchains
Maxim Kuvyrkov [Fri, 2 Sep 2016 13:42:55 +0000 (13:42 +0000)]
Use setrlimit for testing libstdc++ in cross toolchains

* acinclude.m4 (GLIBCXX_CONFIGURE_TESTSUITE): Check for presence of
setrlimit on both native and cross targets.
* configure: Regenerate.

From-SVN: r239955

8 years agoipa-cp.c (ipcp_store_bits_results): Change option name from -fipa-cp-bit to -fipa...
Prathamesh Kulkarni [Fri, 2 Sep 2016 09:22:07 +0000 (09:22 +0000)]
ipa-cp.c (ipcp_store_bits_results): Change option name from -fipa-cp-bit to -fipa-bit-cp.

2016-09-02  Prathamesh Kulkarni  <prathamesh.kulkarni@linaro.org>

* ipa-cp.c (ipcp_store_bits_results): Change option name from
-fipa-cp-bit to -fipa-bit-cp.

From-SVN: r239954

8 years agoPR tree-optimization/71831 - __builtin_object_size poor results with no
Martin Sebor [Fri, 2 Sep 2016 02:14:50 +0000 (02:14 +0000)]
PR tree-optimization/71831 - __builtin_object_size poor results with no

PR tree-optimization/71831 - __builtin_object_size poor results with no
optimization

gcc/testsuite/ChangeLog:

PR tree-optimization/71831
* gcc.dg/builtin-object-size-16.c: New test.
* gcc.dg/builtin-object-size-17.c: New test.

gcc/ChangeLog:

PR tree-optimization/71831
* tree-object-size.h: Return bool instead of the size and add
argument for the size.
* tree-object-size.c (compute_object_offset): Update signature.
(addr_object_size): Same.
(compute_builtin_object_size): Return bool instead of the size
and add argument for the size.  Handle POINTER_PLUS_EXPR when
optimization is disabled.
(expr_object_size): Adjust.
(plus_stmt_object_size): Adjust.
(pass_object_sizes::execute): Adjust.
* builtins.c (fold_builtin_object_size): Adjust.
* doc/extend.texi (Object Size Checking): Update.
* ubsan.c (instrument_object_size): Adjust.

From-SVN: r239953

8 years agoDaily bump.
GCC Administrator [Fri, 2 Sep 2016 00:16:14 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239952

8 years agoc-ada-spec.c (dump_ada_function_declaration): Increase buffer size to guarantee it...
Martin Sebor [Thu, 1 Sep 2016 22:47:49 +0000 (22:47 +0000)]
c-ada-spec.c (dump_ada_function_declaration): Increase buffer size to guarantee it fits the output of the formatted function...

gcc/c-family/ChangeLog:

* c-ada-spec.c (dump_ada_function_declaration): Increase buffer
size to guarantee it fits the output of the formatted function
regardless of its arguments.

gcc/cp/ChangeLog:

* mangle.c: Increase buffer size to guarantee it fits the output
of the formatted function regardless of its arguments.

gcc/go/ChangeLog:

* gofrontend/expressions.cc: Increase buffer size to guarantee
it fits the output of the formatted function regardless of its
arguments.

gcc/java/ChangeLog:

* decl.c (give_name_to_locals): Increase buffer size to guarantee
it fits the output of the formatted function regardless of its
arguments.
* mangle_name.c (append_unicode_mangled_name): Same.

gcc/ChangeLog:

* genmatch.c (parser::parse_expr): Increase buffer size to guarantee
it fits the output of the formatted function regardless of its
arguments.
* gcc/genmodes.c (parser::parse_expr): Same.
* gimplify.c (gimplify_asm_expr): Same.
* passes.c (pass_manager::register_one_dump_file): Same.
* print-tree.c (print_node): Same.

From-SVN: r239949

8 years agors6000: Rename 110 -> VSCR_REGNO
Segher Boessenkool [Thu, 1 Sep 2016 22:28:29 +0000 (00:28 +0200)]
rs6000: Rename 110 -> VSCR_REGNO

* config/rs6000/altivec.md: Use VSCR_REGNO instead of 110 throughout.

From-SVN: r239948

8 years agors6000: Rename 109 -> VRSAVE_REGNO
Segher Boessenkool [Thu, 1 Sep 2016 22:27:20 +0000 (00:27 +0200)]
rs6000: Rename 109 -> VRSAVE_REGNO

* config/rs6000/altivec.md: Use VRSAVE_REGNO instead of 109 throughout.

From-SVN: r239947

8 years agors6000: Rename 74 -> CR6_REGNO
Segher Boessenkool [Thu, 1 Sep 2016 22:26:05 +0000 (00:26 +0200)]
rs6000: Rename 74 -> CR6_REGNO

* config/rs6000/altivec.md: Use CR6_REGNO instead of 74 throughout.
* config/rs6000/vector.md: Ditto.
* config/rs6000/vsx.md: Ditto.

From-SVN: r239946

8 years agore PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)
Jerry DeLisle [Thu, 1 Sep 2016 21:17:42 +0000 (21:17 +0000)]
re PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)

2016-09-01  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

PR libgfortran/77393
* gfortran.dg/fmt_f0_2.f90: Update tests for available kinds.

From-SVN: r239945

8 years agofloat128-builtin.c, [...]: Use dg-add-options ieee.
Uros Bizjak [Thu, 1 Sep 2016 16:04:17 +0000 (18:04 +0200)]
float128-builtin.c, [...]: Use dg-add-options ieee.

* gcc.dg/torture/float128-builtin.c, gcc.dg/torture/float128-floath.c,
gcc.dg/torture/float128-ieee-nan.c, gcc.dg/torture/float128-tg-2.c,
gcc.dg/torture/float128-tg.c, gcc.dg/torture/float128x-builtin.c,
gcc.dg/torture/float128x-floath.c, gcc.dg/torture/float128x-nan.c,
gcc.dg/torture/float128x-tg-2.c, gcc.dg/torture/float128x-tg.c,
gcc.dg/torture/float16-builtin.c, gcc.dg/torture/float16-floath.c,
gcc.dg/torture/float16-nan.c, gcc.dg/torture/float16-tg-2.c,
gcc.dg/torture/float16-tg.c, gcc.dg/torture/float32-builtin.c,
gcc.dg/torture/float32-floath.c, gcc.dg/torture/float32-nan.c,
gcc.dg/torture/float32-tg-2.c, gcc.dg/torture/float32-tg.c,
gcc.dg/torture/float32x-builtin.c, gcc.dg/torture/float32x-floath.c,
gcc.dg/torture/float32x-nan.c, gcc.dg/torture/float32x-tg-2.c,
gcc.dg/torture/float32x-tg.c, gcc.dg/torture/float64-builtin.c,
gcc.dg/torture/float64-floath.c, gcc.dg/torture/float64-nan.c,
gcc.dg/torture/float64-tg-2.c, gcc.dg/torture/float64-tg.c,
gcc.dg/torture/float64x-builtin.c, gcc.dg/torture/float64x-floath.c,
gcc.dg/torture/float64x-nan.c, gcc.dg/torture/float64x-tg-2.c,
gcc.dg/torture/float64x-tg.c: Use dg-add-options ieee.

From-SVN: r239944

8 years agoipa-inline-analysis.c (param_change_prob): Get to the base object first in all cases.
Eric Botcazou [Thu, 1 Sep 2016 15:56:13 +0000 (15:56 +0000)]
ipa-inline-analysis.c (param_change_prob): Get to the base object first in all cases.

* ipa-inline-analysis.c (param_change_prob): Get to the base object
first in all cases.

From-SVN: r239943

8 years agors6000: Use LR_REGNO directly in the save/restore patterns
Segher Boessenkool [Thu, 1 Sep 2016 15:07:54 +0000 (17:07 +0200)]
rs6000: Use LR_REGNO directly in the save/restore patterns

Various patterns use "register_operand" "l" (or "=l") although those
patterns are only created refering to LR_REGNO directly.  This patch
changes those patterns to use the hard regs as well.

* config/rs6000/rs6000.md (*restore_gpregs_<mode>_r11,
*restore_gpregs_<mode>_r12, *restore_gpregs_<mode>_r1,
*return_and_restore_gpregs_<mode>_r11,
*return_and_restore_gpregs_<mode>_r12,
*return_and_restore_gpregs_<mode>_r1,
*return_and_restore_fpregs_<mode>_r11,
*return_and_restore_fpregs_<mode>_r12,
*return_and_restore_fpregs_<mode>_r1): Use the hard register LR_REGNO
directly instead of via the "l" constraint.  Renumber operands.
Fix whitespace.

From-SVN: r239942

8 years agors6000: Use LR_REGNO instead of constant 65
Segher Boessenkool [Thu, 1 Sep 2016 15:04:11 +0000 (17:04 +0200)]
rs6000: Use LR_REGNO instead of constant 65

Many places still use 65 instead of the symbolic constant LR_REGNO.  This
fixes them all (I looked for the string "65" only, in config/rs6000/ only,
I didn't read all code :-) )

I left it in *restore_world because Iain will remove it there soon.

* config/rs6000/altivec.md (*save_world, *save_vregs_<mode>_r11,
save_vregs_<mode>_r12, *restore_vregs_<mode>_r11,
*restore_vregs_<mode>_r12): Use LR_REGNO instead of 65.
* config/rs6000/darwin.md (load_macho_picbase, load_macho_picbase_si,
load_macho_picbase_di, *call_indirect_nonlocal_darwin64,
*call_nonlocal_darwin64, *call_value_indirect_nonlocal_darwin64,
*call_value_nonlocal_darwin64, reload_macho_picbase,
reload_macho_picbase_si, reload_macho_picbase_di): Ditto.
* config/rs6000/rs6000.h (RETURN_ADDR_IN_PREVIOUS_FRAME): Ditto.
* config/rs6000/rs6000.md (*save_gpregs_<mode>_r11,
*save_gpregs_<mode>_r12, *save_gpregs_<mode>_r1,
*save_fpregs_<mode>_r11, *save_fpregs_<mode>_r12,
*save_fpregs_<mode>_r1): Ditto.
* config/rs6000/spe.md (*save_gpregs_spe, *restore_gpregs_spe,
*return_and_restore_gpregs_spe): Ditto.

From-SVN: r239941

8 years agors6000: Fix for AIX, for r239866
Segher Boessenkool [Thu, 1 Sep 2016 15:01:02 +0000 (17:01 +0200)]
rs6000: Fix for AIX, for r239866

This should fix r239866 for AIX.  I missed two patterns that refer to LR
as "register_operand" "l" instead of as reg:P LR_REGNO.

* config/rs6000/rs6000.md
(define_insn "*return_and_restore_fpregs_aix_<mode>_r11"): Delete
the use of the link register.
(define_insn "*return_and_restore_fpregs_aix_<mode>_r1"): Ditto.

From-SVN: r239940

8 years agore PR c/7652 (-Wswitch-break : Warn if a switch case falls through)
Marek Polacek [Thu, 1 Sep 2016 14:59:50 +0000 (14:59 +0000)]
re PR c/7652 (-Wswitch-break : Warn if a switch case falls through)

PR c/7652
gcc/c-family/
* c-common.c (resolve_overloaded_builtin): Fix formatting.  Add
FALLTHRU comments.
gcc/c/
* c-typeck.c (composite_type): Add FALLTHRU comment.
gcc/gcc/cp/
* error.c (dump_type): Fix falls through comment.
(dump_decl): Likewise.
(dump_expr): Likewise.

From-SVN: r239939

8 years agore PR target/72827 (gnat bootstrap broken on powerpc64le-linux-gnu)
Bill Schmidt [Thu, 1 Sep 2016 14:43:55 +0000 (14:43 +0000)]
re PR target/72827 (gnat bootstrap broken on powerpc64le-linux-gnu)

2016-09-01  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
    Michael Meissner <meissner@linux.vnet.ibm.com>

PR target/72827
* config/rs6000/rs6000.c (rs6000_legitimize_address): Avoid
reg+reg addressing for TImode.
(rs6000_legitimate_address_p): Only allow register indirect
addressing for TImode, even without TARGET_QUAD_MEMORY.

Co-Authored-By: Michael Meissner <meissner@linux.vnet.ibm.com>
From-SVN: r239938

8 years agore PR middle-end/77436 (Incorrect constant result for summing loop inserted)
Richard Biener [Thu, 1 Sep 2016 13:38:25 +0000 (13:38 +0000)]
re PR middle-end/77436 (Incorrect constant result for summing loop inserted)

2016-09-01  Richard Biener  <rguenther@suse.de>

PR middle-end/77436
* tree-chrec.c (tree_fold_binomial): Use widest_int, properly
check whether the result fits the desired result type.

* gcc.dg/torture/pr77436.c: New testcase.

From-SVN: r239937

8 years ago* config/nvptx/nvptx.md (cbranch<mode>4): Op 2 can be const.
Nathan Sidwell [Thu, 1 Sep 2016 12:07:16 +0000 (12:07 +0000)]
* config/nvptx/nvptx.md (cbranch<mode>4): Op 2 can be const.

From-SVN: r239934

8 years agoThis patch adds legitimize_address_displacement hook so that stack accesses...
Wilco Dijkstra [Thu, 1 Sep 2016 11:34:49 +0000 (11:34 +0000)]
This patch adds legitimize_address_displacement hook so that stack accesses...

This patch adds legitimize_address_displacement hook so that stack accesses
with large offsets are split into a more efficient sequence.  Unaligned and
TI/TFmode use a 256-byte range, byte and halfword accesses use a 4KB range,
wider accesses use a 16KB range to maximise the available addressing range
and increase opportunities to share the base address.

int f(int x)
{
  int arr[8192];
  arr[4096] = 0;
  arr[6000] = 0;
  arr[7000] = 0;
  arr[8191] = 0;
  return arr[x];
}

Now generates:

  sub   sp, sp, #32768
  add   x1, sp, 16384
  str   wzr, [x1]
  str   wzr, [x1, 7616]
  str   wzr, [x1, 11616]
  str   wzr, [x1, 16380]
  ldr   w0, [sp, w0, sxtw 2]
  add   sp, sp, 32768
  ret

    gcc/
* config/aarch64/aarch64.c (aarch64_legitimize_address_displacement):
New function.
(TARGET_LEGITIMIZE_ADDRESS_DISPLACEMENT): Define.

From-SVN: r239923

8 years agoFix more bogus testsuite failures for avr.
Senthil Kumar Selvaraj [Thu, 1 Sep 2016 11:02:55 +0000 (11:02 +0000)]
Fix more bogus testsuite failures for avr.

gcc/testsuite/

2016-09-01  Senthil Kumar Selvaraj  <senthil_kumar.selvaraj@atmel.com>

        * gcc.dg/pr64252.c: Require int32plus.
        * gcc.dg/pr66299-1.c: Likewise.
        * gcc.dg/pr66299-2.c: Likewise.
        * gcc.dg/torture/20131115-1.c: Skip for avr.

From-SVN: r239921

8 years ago[AArch64] Add ANDS pattern for CMP+ZERO_EXTEND
Kyrylo Tkachov [Thu, 1 Sep 2016 09:03:52 +0000 (09:03 +0000)]
[AArch64] Add ANDS pattern for CMP+ZERO_EXTEND

* config/aarch64/aarch64.md (*ands<mode>_compare0): New pattern.
* config/aarch64/aarch64.c (aarch64_select_cc_mode): Return CC_NZmode
for comparisons of integer ZERO_EXTEND against zero.

* gcc.target/aarch64/ands_3.c: New test.

From-SVN: r239919

8 years agoi386.c (ix86_option_override_internal): Also disable the STV pass if -mstackrealign...
Eric Botcazou [Thu, 1 Sep 2016 07:55:00 +0000 (07:55 +0000)]
i386.c (ix86_option_override_internal): Also disable the STV pass if -mstackrealign is enabled.

* config/i386/i386.c (ix86_option_override_internal): Also disable the
STV pass if -mstackrealign is enabled.

From-SVN: r239916

8 years agoPR c++/57728 - adjust testcase
Jason Merrill [Thu, 1 Sep 2016 01:55:47 +0000 (21:55 -0400)]
PR c++/57728 - adjust testcase

* g++.dg/cpp0x/explicit12.C: Add -save-temps.

From-SVN: r239913

8 years agoC: Fix missing spaces in 'struct' fix-it hints
David Malcolm [Thu, 1 Sep 2016 00:50:54 +0000 (00:50 +0000)]
C: Fix missing spaces in 'struct' fix-it hints

In r237714 I added fix-it hints to the C frontend for missing "struct"
keywords e.g.:

spellcheck-typenames.c:69:1: error: unknown type name ‘foo_t’; use
‘struct’ keyword to refer to the type
 foo_t *foo_ptr;
 ^~~~~
 struct

However when using the (not yet in trunk) option
 -fdiagnostics-generate-patch,
the generated patch is nonsensical:

  -foo_t *foo_ptr;
  +structfoo_t *foo_ptr;

Fix the fix-its by adding a trailing space to each one, giving:

  -foo_t *foo_ptr;
  +struct foo_t *foo_ptr;

gcc/c/ChangeLog:
* c-parser.c (c_parser_declaration_or_fndef): Add trailing space
to the insertion fixits for "struct", "union", and "enum".

From-SVN: r239912

8 years agoDaily bump.
GCC Administrator [Thu, 1 Sep 2016 00:16:16 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239911

8 years agoFix detection of AVX512IFMA in host_detect_local_cpu
Ilya Verbin [Wed, 31 Aug 2016 19:49:35 +0000 (19:49 +0000)]
Fix detection of AVX512IFMA in host_detect_local_cpu

gcc/
* config/i386/driver-i386.c (host_detect_local_cpu): Fix detection of
AVX512IFMA.

From-SVN: r239908

8 years agodiagnostic-show-locus.c: handle fixits on lines outside the regular ranges
David Malcolm [Wed, 31 Aug 2016 18:54:55 +0000 (18:54 +0000)]
diagnostic-show-locus.c: handle fixits on lines outside the regular ranges

The diagnostic_show_locus implementation determines the set
of line spans that need printing based on the ranges within the
rich_location (in layout::calculate_line_spans).

Currently this doesn't take into account fix-it hints, and hence
we fail to print fix-it hints that are on lines outside of
those ranges.

This patch updates the implementation to take fix-it hints into
account when calculating the pertinent line spans, so that such fix-it
hints do get printed.  It also adds some validation, to ensure that
we don't attempt to print fix-its hints affecting a different source
file.

gcc/ChangeLog:
* diagnostic-show-locus.c (class layout): Add field m_fixit_hints.
(layout_range::intersects_line_p): New method.
(test_range_contains_point_for_single_point): Rename to...
(test_layout_range_for_single_point): ...this, and add testing
for layout_range::intersects_line_p.
(test_range_contains_point_for_single_line): Rename to...
(test_layout_range_for_single_line): ...this,  and add testing
for layout_range::intersects_line_p.
(test_range_contains_point_for_multiple_lines): Rename to...
(test_layout_range_for_multiple_lines): ...this,  and add testing
for layout_range::intersects_line_p.
(layout::layout): Populate m_fixit_hints.
(layout::get_expanded_location): Handle the case of a line-span
for a fix-it hint.
(layout::validate_fixit_hint_p): New method.
(get_line_span_for_fixit_hint): New function.
(layout::calculate_line_spans): Add spans for fixit-hints.
(layout::should_print_annotation_line_p): New method.
(layout::print_any_fixits): Drop param "richloc", instead using
validated fixits in m_fixit_hints.  Add "const" to hint pointers.
(diagnostic_show_locus): Avoid printing blank annotation lines.
(selftest::test_diagnostic_context::test_diagnostic_context):
Initialize show_column and start_span.
(selftest::test_diagnostic_context::start_span_cb): New static
function.
(selftest::test_diagnostic_show_locus_fixit_lines): New function.
(selftest::diagnostic_show_locus_c_tests): Update for function
renamings.  Call test_diagnostic_show_locus_fixit_lines.

libcpp/ChangeLog:
* include/line-map.h (class fixit_remove): Remove stray decl.
(fixit_hint::affects_line_p): Make const.
(fixit_insert::affects_line_p): Likewise.
(fixit_replace::affects_line_p): Likewise.
* line-map.c (fixit_insert::affects_line_p): Likewise.
(fixit_replace::affects_line_p): Likewise.

From-SVN: r239906

8 years agore PR tree-optimization/77352 (ICE: verify_ssa failed)
Jakub Jelinek [Wed, 31 Aug 2016 18:42:55 +0000 (20:42 +0200)]
re PR tree-optimization/77352 (ICE: verify_ssa failed)

PR fortran/77352
* trans-openmp.c (gfc_trans_omp_parallel_workshare): Always add a
BIND_EXPR with BLOCK around what gfc_trans_omp_workshare returns.

* gfortran.dg/gomp/pr77352.f90: New test.

From-SVN: r239904

8 years agore PR fortran/77374 (ICE in resolve_omp_atomic, at fortran/openmp.c:3949)
Jakub Jelinek [Wed, 31 Aug 2016 18:42:08 +0000 (20:42 +0200)]
re PR fortran/77374 (ICE in resolve_omp_atomic, at fortran/openmp.c:3949)

PR fortran/77374
* parse.c (parse_omp_oacc_atomic): Copy over cp->ext.omp_atomic
to cp->block->ext.omp_atomic.
* resolve.c (gfc_resolve_blocks): Assert block with one or two
EXEC_ASSIGNs for EXEC_*_ATOMIC.
* openmp.c (resolve_omp_atomic): Don't assert one or two
EXEC_ASSIGNs, instead return quietly for EXEC_NOPs and otherwise
error unexpected statements.

* gfortran.dg/gomp/pr77374.f08: New test.

From-SVN: r239903

8 years agomatch.pd: Revert a * (1 << b) relaxation.
Marc Glisse [Wed, 31 Aug 2016 18:22:58 +0000 (20:22 +0200)]
match.pd: Revert a * (1 << b) relaxation.

2016-08-31  Marc Glisse  <marc.glisse@inria.fr>

PR tree-optimization/73714
gcc/
* match.pd (a * (1 << b)): Revert change from 2016-05-23.
gcc/testsuite/
* gcc.dg/tree-ssa/pr73714.c: New test.

From-SVN: r239902

8 years agore PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)
Jerry DeLisle [Wed, 31 Aug 2016 17:54:32 +0000 (17:54 +0000)]
re PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)

2016-08-31  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

PR libgfortran/77393
* gfortran.dg/fmt_f0_2.f90: New test.

From-SVN: r239901

8 years agore PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)
Jerry DeLisle [Wed, 31 Aug 2016 17:45:26 +0000 (17:45 +0000)]
re PR libfortran/77393 (Revision r237735 changed the behavior of F0.0)

2016-08-31  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

PR libgfortran/77393
* io/write.c (kind_from_size): New function to calculate required buffer
size based on kind type. (select_buffer, select_string): Use new
function. (write_float_0, write_real, write_real_g0, write_complex):
Adjust calls to pass parameters needed by new function.

From-SVN: r239900

8 years agoConstrain std::shared_ptr assignment and resetting
Jonathan Wakely [Wed, 31 Aug 2016 16:57:20 +0000 (17:57 +0100)]
Constrain std::shared_ptr assignment and resetting

* include/bits/shared_ptr.h (_Assignable): New alias template.
(shared_ptr::operator=(const shared_ptr<_Tp1>&))
(shared_ptr::operator=(shared_ptr<_Tp1>&&))
(shared_ptr::operator=(unique_ptr<_Tp1>&&)): Constrain with
_Assignable.
* include/bits/shared_ptr_base.h (_Assignable): New alias template.
(__shared_ptr::operator=(const __shared_ptr<_Tp1>&))
(__shared_ptr::operator=(__shared_ptr<_Tp1>&&))
(__shared_ptr::operator=(unique_ptr<_Tp1>&&)): Constrain with
_Assignable.
(__shared_ptr::reset(_Tp1*), __shared_ptr::reset(_Tp1*, _Deleter))
(__shared_ptr::reset(_Tp1*, _Deleter, _Alloc)): Constrain with
_Convertible.
* testsuite/20_util/shared_ptr/cons/43820_neg.cc: Change dg-error to
match on any line.
* testsuite/20_util/shared_ptr/cons/void_neg.cc: Likewise.
* testsuite/20_util/shared_ptr/assign/sfinae.cc: New test.
* testsuite/20_util/shared_ptr/assign/shared_ptr_neg.cc: Update
expected errors. Remove unnecessary code.
* testsuite/20_util/shared_ptr/modifiers/reset_sfinae.cc: New test.

From-SVN: r239898

8 years agoMove comparison object in map/set move assignment
Jonathan Wakely [Wed, 31 Aug 2016 16:57:14 +0000 (17:57 +0100)]
Move comparison object in map/set move assignment

* include/bits/stl_tree.h (_Rb_tree::operator=(_Rb_tree&&)): Move
comparison object.
* testsuite/23_containers/set/move_comparison.cc: New test.

From-SVN: r239897

8 years agoAdd test accidentally not added in revision r239121
Jonathan Wakely [Wed, 31 Aug 2016 16:57:09 +0000 (17:57 +0100)]
Add test accidentally not added in revision r239121

* testsuite/20_util/enable_shared_from_this/members/weak_from_this.cc:
New test.

From-SVN: r239896

8 years agoruntime: make gsignal stack at least SIGSTKSZ bytes
Ian Lance Taylor [Wed, 31 Aug 2016 13:59:03 +0000 (13:59 +0000)]
runtime: make gsignal stack at least SIGSTKSZ bytes

    The default stack size for the gsignal goroutine, 32K, is not enough on
    ia64.  Make sure that the stack size is at least SIGSTKSZ.

    Reviewed-on: https://go-review.googlesource.com/28224

From-SVN: r239894

8 years agoselftest.c: avoid explicit "selftest::" qualifiers
David Malcolm [Wed, 31 Aug 2016 13:22:43 +0000 (13:22 +0000)]
selftest.c: avoid explicit "selftest::" qualifiers

gcc/ChangeLog:
* selftest.c: Move "namespace selftest {" to top of file,
removing explicit "selftest::" qualifiers throughout.

From-SVN: r239892

8 years ago[x86] Don't use builtins for unaligned load/store
Marc Glisse [Wed, 31 Aug 2016 11:56:37 +0000 (13:56 +0200)]
[x86] Don't use builtins for unaligned load/store

2016-08-31  Marc Glisse  <marc.glisse@inria.fr>

gcc/
* config/i386/avx512fintrin.h (__m512_u, __m512i_u, __m512d_u):
New types.
(_mm512_loadu_pd, _mm512_storeu_pd, _mm512_loadu_ps,
_mm512_storeu_ps, _mm512_loadu_si512, _mm512_storeu_si512):
Replace builtin with vector extension.
* config/i386/avxintrin.h (__m256_u, __m256i_u, __m256d_u):
New types.
(_mm256_loadu_pd, _mm256_storeu_pd, _mm256_loadu_ps,
_mm256_storeu_ps, _mm256_loadu_si256, _mm256_storeu_si256):
Replace builtin with vector extension.
* config/i386/emmintrin.h (__m128i_u, __m128d_u): New types.
(_mm_loadu_pd, _mm_storeu_pd, _mm_loadu_si128, _mm_storeu_si128):
Replace builtin with vector extension.
* config/i386/xmmintrin.h (__m128_u): New type.
(_mm_loadu_ps, _mm_storeu_ps): Replace builtin with vector extension.
(_mm_load_ps, _mm_store_ps): Simplify.

gcc/testsuite/
* gcc.target/i386/pr59539-2.c: Adapt options.
* gcc.target/i386/avx512f-vmovdqu32-1.c: Relax expected asm.

From-SVN: r239889

8 years ago* config/arm/arm.c (thumb1_size_rtx_costs) <SET>: Add missing guard.
Eric Botcazou [Wed, 31 Aug 2016 10:09:00 +0000 (10:09 +0000)]
* config/arm/arm.c (thumb1_size_rtx_costs) <SET>: Add missing guard.

From-SVN: r239888

8 years agore PR testsuite/77418 (SELECT TYPE associating entity can be (de)allocated)
Paul Thomas [Wed, 31 Aug 2016 07:50:44 +0000 (07:50 +0000)]
re PR testsuite/77418 (SELECT TYPE associating entity can be (de)allocated)

2016-08-31  Paul Thomas  <pault@gcc.gnu.org>

PR fortran/77418
* gfortran.dg/dtio_5.f90: Remove the non-standard deallocation
of the associate entity and replace with a pointer to the
intended item on the stack.

From-SVN: r239881

8 years ago[multiple changes]
Paul Thomas [Wed, 31 Aug 2016 05:36:22 +0000 (05:36 +0000)]
[multiple changes]

2016-08-31  Paul Thomas  <pault@gcc.gnu.org>
Jerry DeLisle  <jvdelisle@gcc.gnu.org>

PR fortran/48298

* decl.c (access_attr_decl): Include case INTERFACE_DTIO as
appropriate.
* gfortran.h : Add INTRINSIC_FORMATTED and
INTRINSIC_UNFORMATTED to gfc_intrinsic_op. Add INTERFACE_DTIO
to interface type. Add new enum 'dtio_codes'. Add bitfield
'has_dtio_procs' to symbol_attr. Add prototypes
'gfc_check_dtio_interfaces' and 'gfc_find_specific_dtio_proc'.
* interface.c (dtio_op): New function.
(gfc_match_generic_spec): Match generic DTIO interfaces.
(gfc_match_interface): Treat DTIO interfaces in the same way as
(gfc_current_interface_head): Add INTERFACE_DTIO appropriately.
(check_dtio_arg_TKR_intent): New function.
(check_dtio_interface1): New function.
(gfc_check_dtio_interfaces): New function.
(gfc_find_specific_dtio_proc): New function.
* io.c : Add FMT_DT to format_token.
(format_lex): Handle DTIO formatting.
* match.c (gfc_op2string): Add DTIO operators.
* resolve.c (derived_inaccessible): Ignore pointer components
to enclosing derived type.
(resolve_transfer): Resolve transfers that involve DTIO.
procedures. Find the specific subroutine for the transfer and
use its existence to over-ride some of the constraints on
derived types. If the transfer is recursive, require that the
subroutine be so qualified.
(dtio_procs_present): New function.
(resolve_fl_namelist): Remove inhibition of polymorphic objects
in namelists if DTIO read and write subroutines exist. Likewise
for derived types.
(resolve_types): Invoke 'gfc_verify_dtio_procedures'.
* symbol.c : Set 'dtio_procs' using 'minit'.
* trans-decl.c (gfc_finish_var_decl): If a derived-type/class
object is associated with DTIO procedures, make it TREE_STATIC.
* trans-expr.c (gfc_get_vptr_from_expr): If the expression
drills down to a PARM_DECL, extract the vptr correctly.
(gfc_conv_derived_to_class): Check 'info' in the test for
'useflags'. If the se expression exists and is a pointer, use
it as the class _data.
* trans-io.c : Add IOCALL_X_DERIVED to iocall and the function
prototype. Likewise for IOCALL_SET_NML_DTIO_VAL.
(set_parameter_tree): Renamed from 'set_parameter_const', now
returns void and has new tree argument. Calls modified to match
new interface.
(transfer_namelist_element): Transfer DTIO procedure pointer
and vpointer using the new function IOCALL_SET_NML_DTIO_VAL.
(get_dtio_proc): New function.
(transfer_expr): Add new argument for the vptr field of class
objects. Add the code to call the specific DTIO proc, convert
derived types to class and call IOCALL_X_DERIVED.
(trans_transfer): Add BT_CLASS to structures for treatment by
the scalarizer. Obtain the vptr for the dynamic type, both for
scalar and array transfer.

2016-08-31  Jerry DeLisle  <jvdelisle@gcc.gnu.org>
Paul Thomas  <pault@gcc.gnu.org>

PR libgfortran/48298
* gfortran.map : Flag _st_set_nml_dtio_var and
_gfortran_transfer_derived.
* io/format.c (format_lex): Detect DTIO formatting.
(parse_format_list): Parse the DTIO format.
(next_format): Include FMT_DT.
* io/format.h : Likewise. Add structure 'udf' to structure
'fnode' to carry the IOTYPE string and the 'vlist'.
* io/io.h : Add prototypes for the two types of DTIO subroutine
and a typedef for gfc_class. Also, add to 'namelist_type'
fields for the pointer to the DTIO procedure and the vtable.
Add fields to struct st_parameter_dt for pointers to the two
types of DTIO subroutine. Add to gfc_unit DTIO specific fields.
(internal_proto): Add prototype for 'read_user_defined' and
'write_user_defined'.
* io/list_read.c (check_buffers): Use the 'current_unit' field.
(unget_char): Likewise.
(eat_spaces): Likewise.
(list_formatted_read_scalar): For case BT_CLASS, call the DTIO
procedure.
(nml_get_obj_data): Likewise when DTIO procedure is present,.
* io/transfer.c : Export prototypes for 'transfer_derived' and
'transfer_derived_write'.
(unformatted_read): For case BT_CLASS, call the DTIO procedure.
(unformatted_write): Likewise.
(formatted_transfer_scalar_read): Likewise.
(formatted_transfer_scalar_write: Likewise.
(transfer_derived): New function.
(data_transfer_init): Set last_char if no child_dtio.
(finalize_transfer): Return if child_dtio set.
(st_write_done): Add condition for child_dtio not set.
Add extra arguments for st_set_nml_var prototype.
(set_nml_var): New function that contains the contents of the
old version of st_set_nml_var. Also sets the 'dtio_sub' and
'vtable' fields of the 'nml' structure.
(st_set_nml_var): Now just calls set_nml_var with 'dtio_sub'
and 'vtable' NULL.
(st_set_nml_dtio_var): New function that calls set_nml_var.
* io/unit.c (get_external_unit): If the found unit child_dtio
is non zero, don't do any mutex locking/unlocking.  Just
return the unit.
* io/unix.c (tempfile_open): Revert to C style comment.
* io/write.c (list_formatted_write_scalar): Do the DTIO call.
(nml_write_obj): Add BT_CLASS and do the DTIO call.

2016-08-31  Jerry DeLisle  <jvdelisle@gcc.gnu.org>
Paul Thomas  <pault@gcc.gnu.org>

PR fortran/48298
* gfortran.dg/dtio_1.f90: New test.
* gfortran.dg/dtio_2.f90: New test.
* gfortran.dg/dtio_3.f90: New test.
* gfortran.dg/dtio_4.f90: New test.
* gfortran.dg/dtio_5.f90: New test.
* gfortran.dg/dtio_6.f90: New test.
* gfortran.dg/dtio_7.f90: New test.
* gfortran.dg/dtio_8.f90: New test.
* gfortran.dg/dtio_9.f90: New test.
* gfortran.dg/dtio_10.f90: New test.

From-SVN: r239880

8 years agoRemove arbitrary limits from rich_location
David Malcolm [Wed, 31 Aug 2016 00:35:01 +0000 (00:35 +0000)]
Remove arbitrary limits from rich_location

This patch eliminates the hard-coded limits within rich_location
(up to 3 ranges, up to 2 fixits).  The common case is still
handled by embedding the values inside rich_location - it only
uses dynamic allocation if these limits are exceeded, so
creation of rich_location instances on the stack should still
be fast.  This is implemented via a new container class,
semi_embedded_vec <T, N>.

gcc/ChangeLog:
* diagnostic-show-locus.c (colorizer::begin_state): Support more
than 3 ranges per diagnostic by alternating between color 1 and
color 2.
(layout::layout): Replace use of rich_location::MAX_RANGES
with richloc->get_num_locations ().
(layout::calculate_line_spans): Replace use of
rich_location::MAX_RANGES with m_layout_ranges.length ().
(layout::print_annotation_line): Handle arbitrary numbers of
ranges in caret-printing by defaulting to '^'.
(selftest::test_one_liner_many_fixits): New function.
(test_diagnostic_show_locus_one_liner): Call it.
* diagnostic.c (diagnostic_initialize): Update for renaming
of rich_location::MAX_RANGES to
rich_location::STATICALLY_ALLOCATED_RANGES.
* diagnostic.h (struct diagnostic_context): Likewise.

gcc/testsuite/ChangeLog:
* gcc.dg/plugin/diagnostic-test-show-locus-bw.c
(test_many_nested_locations): New function.
* gcc.dg/plugin/diagnostic_plugin_test_show_locus.c
(test_show_locus): Handle "test_many_nested_locations".

libcpp/ChangeLog:
* include/line-map.h (class semi_embedded_vec): New class.
(semi_embedded_vec<T, NUM_EMBEDDED>::semi_embedded_vec): New ctor.
(semi_embedded_vec<T, NUM_EMBEDDED>::~semi_embedded_vec): New
dtor.
(semi_embedded_vec<T, NUM_EMBEDDED>::operator[]): New methods.
(semi_embedded_vec<T, NUM_EMBEDDED>::push): New method.
(semi_embedded_vec<T, NUM_EMBEDDED>::truncate): New method.
(rich_location::get_num_locations): Reimplement in terms of
m_ranges.
(rich_location::get_range): Make non-inline.
(rich_location::get_num_fixit_hints): Reimplement in terms of
m_fixit_hints.
(rich_location::add_fixit): New function.
(rich_location::MAX_RANGES): Rename to...
(rich_location::STATICALLY_ALLOCATED_RANGES): ...this.
(rich_location::MAX_FIXIT_HINTS): Rename to...
(rich_location::STATICALLY_ALLOCATED_RANGES): ...this, and make
private.
(rich_location::m_num_ranges): Eliminate in favor of...
(rich_location::m_ranges): ...this, converting from a fixed-size
array to a semi_embedded_vec.
(rich_location::m_num_fixit_hints): Eliminate in favor of...
(rich_location::m_fixit_hints): ...this, converting from a
fixed-size array to a semi_embedded_vec.
* line-map.c (rich_location::rich_location): Update for above
changes.
(rich_location::~rich_location): Likewise.
(rich_location::get_loc): Likewise.
(rich_location::get_range): New methods.
(rich_location::add_range): Update for above changes.
(rich_location::set_range): Likewise.
(rich_location::add_fixit_insert): Likewise.
(rich_location::add_fixit_replace): Likewise.
(rich_location::get_last_fixit_hint): Likewise.
(rich_location::reject_impossible_fixit): Likewise.
(rich_location::add_fixit): New method.

From-SVN: r239879

8 years agoDaily bump.
GCC Administrator [Wed, 31 Aug 2016 00:16:14 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239878

8 years agoselftest: split out named_temp_file from temp_source_file
David Malcolm [Wed, 31 Aug 2016 00:07:38 +0000 (00:07 +0000)]
selftest: split out named_temp_file from temp_source_file

Split out a new base class for temp_source_file, named_temp_file,
moving the deletion to the base class dtor, so that we can write
out temporary files in other ways in selftests.

gcc/ChangeLog:
* selftest.c (selftest::named_temp_file::named_temp_file): New
ctor.
(selftest::temp_source_file::~temp_source_file): Move to...
(selftest::named_temp_file::~named_temp_file): ...here.
(selftest::test_named_temp_file): New function.
(selftest::selftest_c_tests): Call test_named_temp_file.
* selftest.h (class named_temp_file): New class.
(class temp_source_file): Convert to a subclass of named_temp_file.

From-SVN: r239875

8 years agoC++: add fixit for '>>' template error
David Malcolm [Tue, 30 Aug 2016 21:21:42 +0000 (21:21 +0000)]
C++: add fixit for '>>' template error

gcc/cp/ChangeLog:
* parser.c (cp_parser_enclosed_template_argument_list): Add fix-it
hint to ">>" within nested template argument list error.

gcc/testsuite/ChangeLog:
* g++.dg/template/double-greater-than-fixit.C: New test case.

From-SVN: r239873

8 years agoruntime: use -fgo-c-header to build C header file
Ian Lance Taylor [Tue, 30 Aug 2016 21:07:47 +0000 (21:07 +0000)]
runtime: use -fgo-c-header to build C header file

    Use the new -fgo-c-header option to build a header file for the Go
    runtime code in libgo/go/runtime, and use the new header file in the C
    runtime code in libgo/runtime.  This will ensure that the Go code and C
    code share the same data structures as we convert the runtime from C to
    Go.

    The new file libgo/go/runtime/runtime2.go is copied from the Go 1.7
    release, and then edited to remove unnecessary data structures and
    modify others for use with libgo.

    The new file libgo/go/runtime/mcache.go is an initial version of the
    same files in the Go 1.7 release, and will be replaced by the Go 1.7
    file when we convert to the new memory allocator.

    The new file libgo/go/runtime/type.go describes the gccgo version of the
    reflection data structures, and replaces the Go 1.7 runtime file which
    describes the gc version of those structures.

    Using the new header file means changing a number of struct fields to
    use Go naming conventions (that is, no underscores) and to rename
    constants to have a leading underscore so that they are not exported
    from the Go package.  These names were updated in the C code.

    The C code was also changed to drop the thread-local variable m, as was
    done some time ago in the gc sources.  Now the m field is always
    accessed using g->m, where g is the single remaining thread-local
    variable.  This in turn required some adjustments to set g->m correctly
    in all cases.

    Also pass the new -fgo-compiling-runtime option when compiling the
    runtime package, although that option doesn't do anything yet.

    Reviewed-on: https://go-review.googlesource.com/28051

From-SVN: r239872

8 years agore PR libstdc++/77395 (std::is_constructible is false for type constructible via...
Ville Voutilainen [Tue, 30 Aug 2016 18:46:11 +0000 (21:46 +0300)]
re PR libstdc++/77395 (std::is_constructible is false for type constructible via implicit conversion operator affecting std::tuple)

PR libstdc++/77395
* include/std/type_traits (is_constructible): Forward-declare...
(__is_base_to_derived_ref): ...and use here.
* testsuite/20_util/declval/requirements/1_neg.cc: Adjust.
* testsuite/20_util/is_constructible/77395.cc: New.
* testsuite/20_util/make_signed/requirements/typedefs_neg.cc: Adjust.
* testsuite/20_util/make_unsigned/requirements/typedefs_neg.cc:
Likewise.
* testsuite/20_util/tuple/77395.cc: New.

From-SVN: r239870

8 years ago2.cc: Move dg-do run directive above dg-require-namedlocale directive.
Uros Bizjak [Tue, 30 Aug 2016 16:59:39 +0000 (18:59 +0200)]
2.cc: Move dg-do run directive above dg-require-namedlocale directive.

* testsuite/22_locale/time_get/get/char/2.cc: Move dg-do run
directive above dg-require-namedlocale directive.
* testsuite/22_locale/time_get/get/wchar_t/2.cc: Ditto.
* testsuite/27_io/manipulators/extended/get_time/char/2.cc: Ditto.
* testsuite/27_io/manipulators/extended/get_time/wchar_t/2.cc: Ditto.
* testsuite/27_io/manipulators/extended/put_time/char/2.cc: Ditto.
* testsuite/27_io/manipulators/extended/put_time/wchar_t/2.cc: Ditto.

From-SVN: r239867

8 years agors6000: Don't emit a use of LR in returns and sibcalls
Segher Boessenkool [Tue, 30 Aug 2016 16:30:01 +0000 (18:30 +0200)]
rs6000: Don't emit a use of LR in returns and sibcalls

The exit block (to which every return artificially jumps) already has
a use of LR.  The LR use in all returns and sibcalls is an anachronism,
probably made unnecessary by the dataflow merge.  The simple_returns
that shrink-wrapping generates also do not have such a use.  Newer
backends do not do this either it seems.

With this use removed, a normal return is no longer a parallel but just
a return insn, and cfgcleanup then can transform conditional jumps to
those into conditional returns.

This splits the return emission code with restoring_FPRs_inline from
that without it; this is simpler code, fewer lines, and less indentation.

The return_internal_<mode> pattern can now be deleted since nothing uses
it anymore.

* config/rs6000/rs6000.c (rs6000_emit_epilogue): Do not emit
USEs of LR_REGNO in returns and sibcalls.
(rs6000_output_mi_thunk): Similar.
(rs6000_sibcall_aix): Similar.
* config/rs6000/rs6000.md (sibcall, sibcall_value, sibcall_local32,
sibcall_local64, sibcall_value_local32, sibcall_value_local64,
sibcall_nonlocal_sysv<mode>, sibcall_value_nonlocal_sysv<mode>):
Remove the USE of LR_REGNO from the patterns as well.  Delete an
obsolete comment.
(return_internal_<mode>): Delete.

From-SVN: r239866

8 years ago[PATCH][Aarch64][gcc] Fix vld2/3/4 on big endian systems
Tamar Christina [Tue, 30 Aug 2016 15:40:19 +0000 (15:40 +0000)]
[PATCH][Aarch64][gcc] Fix vld2/3/4 on big endian systems

On behalf of Tamar Christina  <tamar.christina@arm.com>

gcc/

* gcc/config/aarch64/aarch64-simd.md
(aarch64_ld2<mode>_dreg_le): New.
(aarch64_ld2<mode>_dreg_be): New.
(aarch64_ld2<mode>_dreg): Removed.
(aarch64_ld3<mode>_dreg_le): New.
(aarch64_ld3<mode>_dreg_be): New.
(aarch64_ld3<mode>_dreg): Removed.
(aarch64_ld4<mode>_dreg_le): New.
(aarch64_ld4<mode>_dreg_be): New.
(aarch64_ld4<mode>_dreg): Removed.
(aarch64_ld<VSTRUCT:nregs><VDC:mode>): Wrapper around _le, _be.

From-SVN: r239865

8 years agogfortran.texi: Fix typo in STRUCTURE documentation.
Fritz Reese [Tue, 30 Aug 2016 14:33:00 +0000 (14:33 +0000)]
gfortran.texi: Fix typo in STRUCTURE documentation.

2016-08-30  Fritz Reese  <fritzoreese@gmail.com>

* gcc/fortran/gfortran.texi: Fix typo in STRUCTURE documentation.

From-SVN: r239862

8 years agorich_location: add convenience overloads for adding fix-it hints
David Malcolm [Tue, 30 Aug 2016 13:54:48 +0000 (13:54 +0000)]
rich_location: add convenience overloads for adding fix-it hints

Adding a fix-it hint to a diagnostic usually follows one of these
patterns:
(a) an insertion fix-its, with the insertion at the primary caret location
(b) a removals/replacements, affecting the range of the primary location

(other cases are possible, e.g. multiple fix-its, and affecting other
locations, but these are the common ones)

Given these common cases, this patch adds overloads of the rich_location
methods for adding fix-it hints, so that the location information can
be omitted if it matches that of the primary location within the
rich_location.

Similarly when adding "remove" and "replace" fix-it hints to a diagnostic,
it's tedious to have to extract the source_range from a location_t
(aka source_location).  To make this more convenient, this patch
adds overload of the rich_location::add_fixit_remove/replace methods,
accepting a source_location directly.

The patch updates the various in-tree users of fix-it hints to use
the new simpler API where appropriate.  I didn't touch the case where
there are multiple fix-its in one rich_location, as it seems better to
be more explicit about locations for this case (adding a pair of parens
in warn_logical_not_parentheses).

The above makes the gcc_rich_location::add_fixit_misspelled_id overload
taking a const char * rather redundant, so I eliminated it.

gcc/c/ChangeLog:
* c-decl.c (implicit_decl_warning): Use add_fixit_replace
rather than add_fixit_misspelled_id.
(undeclared_variable): Likewise.
* c-parser.c (c_parser_declaration_or_fndef): Likewise.  Remove
now-redundant "here" params from add_fixit_insert method calls.
(c_parser_parameter_declaration): Likewise.
* c-typeck.c (build_component_ref): Remove now-redundant range
param from add_fixit_replace method calls.

gcc/cp/ChangeLog:
* name-lookup.c (suggest_alternatives_for): Use add_fixit_replace
rather than add_fixit_misspelled_id.
* parser.c (cp_parser_diagnose_invalid_type_name): Likewise.

gcc/ChangeLog:
* diagnostic-show-locus.c (test_one_liner_fixit_insert): Remove
redundant location param.
(test_one_liner_fixit_remove): Likewise.
(test_one_liner_fixit_replace): Likewise.
(test_one_liner_fixit_replace_equal_secondary_range): Likewise.
* gcc-rich-location.c
(gcc_rich_location::add_fixit_misspelled_id): Eliminate call to
get_range_from_loc.  Drop overload taking a const char *.
* gcc-rich-location.h
(gcc_rich_location::add_fixit_misspelled_id): Drop overload taking
a const char *.

libcpp/ChangeLog:
* include/line-map.h (rich_location::add_fixit_insert): Add
comments.  Add overload omitting the source_location param.
(rich_location::add_fixit_remove): Add comments.  Add overloads
omitting the range, and accepting a source_location.
(rich_location::add_fixit_replace): Likewise.
* line-map.c (rich_location::add_fixit_insert): Add comments.  Add
overload omitting the source_location param.
(rich_location::add_fixit_remove): Add comments.  Add overloads
omitting the range, and accepting a source_location.
(rich_location::add_fixit_replace): Likewise.

From-SVN: r239861

8 years agoreport supported function classes correctly on *-musl
Szabolcs Nagy [Tue, 30 Aug 2016 10:28:06 +0000 (10:28 +0000)]
report supported function classes correctly on *-musl

gcc/
* config/linux.c (linux_libc_has_function): Return true on musl.

From-SVN: r239860

8 years agodisable ifunc on *-musl by default
Szabolcs Nagy [Tue, 30 Aug 2016 10:26:22 +0000 (10:26 +0000)]
disable ifunc on *-musl by default

gcc/
* config.gcc (*-*-*musl*): Disable gnu-indirect-function.

From-SVN: r239859

8 years agopostreload-gcse.c (bb_has_well_behaved_predecessors): Tweak criterion used for abnorm...
Eric Botcazou [Tue, 30 Aug 2016 10:10:26 +0000 (10:10 +0000)]
postreload-gcse.c (bb_has_well_behaved_predecessors): Tweak criterion used for abnormal egdes.

* postreload-gcse.c (bb_has_well_behaved_predecessors): Tweak criterion
used for abnormal egdes.

From-SVN: r239858

8 years agore PR tree-optimization/69047 (memcpy is not as optimized as union is)
Richard Biener [Tue, 30 Aug 2016 09:22:17 +0000 (09:22 +0000)]
re PR tree-optimization/69047 (memcpy is not as optimized as union is)

2016-08-30  Richard Biener  <rguenther@suse.de>

PR tree-optimization/69047
* gcc.dg/pr69047.c: Fix byte-order check.

From-SVN: r239857

8 years agore PR tree-optimization/72866 (Compile time hog w/ -O3 (-Ofast))
Jakub Jelinek [Tue, 30 Aug 2016 06:54:02 +0000 (08:54 +0200)]
re PR tree-optimization/72866 (Compile time hog w/ -O3 (-Ofast))

PR tree-optimization/72866
* tree-vect-patterns.c (search_type_for_mask): Turn into
a small wrapper, move all code to ...
(search_type_for_mask_1): ... this new function.  Add caching
and adjust recursive calls.

* gcc.dg/vect/pr72866.c: New test.

From-SVN: r239856

8 years agore PR c++/77363 (Missing debug information in DWARF)
Jakub Jelinek [Tue, 30 Aug 2016 06:46:38 +0000 (08:46 +0200)]
re PR c++/77363 (Missing debug information in DWARF)

PR debug/77363
* dwarf2out.c (modified_type_die): Use lookup_type_die (type)
instead of lookup_type_die (type_main_variant (type)) even for array
types.

* g++.dg/debug/dwarf2/pr77363.C: New test.

From-SVN: r239855

8 years agore PR middle-end/77377 (c-c++-common/pr59037.c ICEs with -fpic -msse on i686)
Jakub Jelinek [Tue, 30 Aug 2016 06:45:56 +0000 (08:45 +0200)]
re PR middle-end/77377 (c-c++-common/pr59037.c ICEs with -fpic -msse on i686)

PR middle-end/77377
* simplify-rtx.c (avoid_constant_pool_reference): For out of bounds
constant pool reference return x instead of c.

* gcc.target/i386/pr77377.c: New test.

From-SVN: r239854

8 years agore PR debug/77389 (FAIL: g++.dg/debug/dwarf2/template-params-12f.C -std=gnu++11...
Jakub Jelinek [Tue, 30 Aug 2016 06:44:43 +0000 (08:44 +0200)]
re PR debug/77389 (FAIL: g++.dg/debug/dwarf2/template-params-12f.C  -std=gnu++11  scan-assembler-times DIE \\\\([^\\n]*\\\\) DW_TAG_template_value_param ...)

PR debug/77389
* g++.dg/debug/dwarf2/template-params-12f.C: Pass -gno-strict-dwarf
in dg-options.

From-SVN: r239853

8 years agocompiler: add -fgo-c-header=FILE to create a C header
Ian Lance Taylor [Tue, 30 Aug 2016 03:27:43 +0000 (03:27 +0000)]
compiler: add -fgo-c-header=FILE to create a C header

    The new -fgo-c-header=FILE option will write a C header file defining
    all the struct types and numeric const values in package scope.  This
    will be used when building the Go runtime package (libgo/go/runtime) to
    generate a C header file that may be included by the C code in the C
    runtime package (libgo/runtime).

    This will ensure that the Go code and C code are working with the same
    data structures as we convert the runtime from C to Go to upgrade to the
    current GC runtime, notably the concurrent garbage collector.

    Reviewed-on: https://go-review.googlesource.com/28000

* lang.opt (fgo-c-header, fgo-compiling-runtime): New options.
* go-c.h (struct go_create_gogo_args): Define.
(go_create_gogo): Change declaration to take struct pointer.
* go-lang.c (go_c_header): New static variable.
(go_langhook_init): Update call to go_create_gogo.
* gccgo.texi (Invoking gccgo): Document -fgo-c-header and
-fgo-compiling-runtime.

From-SVN: r239852

8 years agoEradicate MQ some more
Segher Boessenkool [Tue, 30 Aug 2016 02:46:07 +0000 (04:46 +0200)]
Eradicate MQ some more

Nothing uses MQ anymore, but it still shows up in all the dump files.
This patch removes it from CALL_REALLY_USED_REGISTERS so that that does
not happen anymore (it is still a fixed register, there should be no
functional change).

* config/rs6000/rs6000.h (CALL_REALLY_USED_REGISTERS): Do not
include MQ.

From-SVN: r239851

8 years agoDaily bump.
GCC Administrator [Tue, 30 Aug 2016 00:16:19 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239850

8 years agopr77372.f90: Moved to ...
Steven G. Kargl [Mon, 29 Aug 2016 23:19:25 +0000 (23:19 +0000)]
pr77372.f90: Moved to ...

2016-08-29  Steven G. Kargl  <kargl@gcc.gnu.org>

* gfortran.dg/pr77372.f90: Moved to ...
* gfortran.dg/ieee/pr77372.f90: here.

From-SVN: r239847

8 years agoinput.c: fix the comment for test_make_location_nonpure_range_endpoints
David Malcolm [Mon, 29 Aug 2016 20:52:10 +0000 (20:52 +0000)]
input.c: fix the comment for test_make_location_nonpure_range_endpoints

gcc/ChangeLog:
* input.c
(selftest::test_make_location_nonpure_range_endpoints): Fix
header comment.

From-SVN: r239844

8 years agoAllow the use of ad-hoc locations for fix-it hints
David Malcolm [Mon, 29 Aug 2016 20:42:57 +0000 (20:42 +0000)]
Allow the use of ad-hoc locations for fix-it hints

Currently the fix-it validator rejects ad-hoc locations.
Fix this by calling get_pure_location on the input locations to
add_fixit_insert/replace.  Doing so requires moving get_pure_location
from gcc to libcpp.

gcc/ChangeLog:
* diagnostic-show-locus.c
(selftest::test_one_liner_fixit_validation_adhoc_locations): New
function.
(selftest::test_diagnostic_show_locus_one_liner): Call it.
* input.c (get_pure_location): Move to libcpp/line-map.c.
* input.h (get_pure_location): Convert decl to an inline function
calling implementation in libcpp.

libcpp/ChangeLog:
* include/line-map.h (get_pure_location): New decl.
* line-map.c (get_pure_location): Move here, from gcc/input.c, adding
a line_maps * param.
(rich_location::add_fixit_insert): Call get_pure_location on "where".
(rich_location::add_fixit_replace): Call get_pure_location on the
end-points.

From-SVN: r239843

8 years agovec-init-4.c: New runtime tests for various vector short/char initializations.
Michael Meissner [Mon, 29 Aug 2016 19:38:07 +0000 (19:38 +0000)]
vec-init-4.c: New runtime tests for various vector short/char initializations.

2016-08-29  Michael Meissner  <meissner@linux.vnet.ibm.com>

* gcc.target/powerpc/vec-init-4.c: New runtime tests for various
vector short/char initializations.
* gcc.target/powerpc/vec-init-5.c: Likewise.
* gcc.target/powerpc/vec-init-6.c: New compile time test for
vector initialization optimizations.
* gcc.target/powerpc/vec-init-7.c: Likewise.
* gcc.target/powerpc/vec-init-8.c: New runtime tests for various
vector float/double initializations.
* gcc.target/powerpc/vec-init-9.c: Likewise.

From-SVN: r239837

8 years agoconfigure.ac (nvptx-*): Hardwire newlib.
Nathan Sidwell [Mon, 29 Aug 2016 19:09:57 +0000 (19:09 +0000)]
configure.ac (nvptx-*): Hardwire newlib.

* configure.ac (nvptx-*): Hardwire newlib.
* configure: Rebuilt.

From-SVN: r239836

8 years agore PR target/77403 (assemble failure of vinserti64x4 with -masm=intel)
Uros Bizjak [Mon, 29 Aug 2016 18:58:10 +0000 (20:58 +0200)]
re PR target/77403 (assemble failure of vinserti64x4 with -masm=intel)

PR target/77403
* config/i386/sse.md (vec_set_lo_<mode><mask_name>): Fix assembler
template for intel asm dialect.
(vec_set_hi_<mode><mask_name>): Ditto.

testsuite/ChangeLog:

PR target/77403
* gcc.target/i386/pr77403.c: New test.

From-SVN: r239834

8 years agore PR c/77292 (Spurious warning: logical not is only applied to the left hand side...
Marek Polacek [Mon, 29 Aug 2016 18:13:13 +0000 (18:13 +0000)]
re PR c/77292 (Spurious warning: logical not is only applied to the left hand side of comparison)

PR c/77292
* c-common.c (warn_logical_not_parentheses): Don't warn for
a comparison or a logical operator.

* c-c++-common/Wlogical-not-parentheses-1.c: New test.

From-SVN: r239833

8 years agoselftest.h: mark failure functions with ATTRIBUTE_NORETURN.
David Malcolm [Mon, 29 Aug 2016 17:23:14 +0000 (17:23 +0000)]
selftest.h: mark failure functions with ATTRIBUTE_NORETURN.

gcc/ChangeLog:
* selftest.h (selftest::fail): Add ATTRIBUTE_NORETURN.
(selftest::fail_formatted): Likewise.

From-SVN: r239832

8 years agomake_location: ensure end-points are pure locations
David Malcolm [Mon, 29 Aug 2016 17:10:23 +0000 (17:10 +0000)]
make_location: ensure end-points are pure locations

gcc/ChangeLog:
* input.c (make_location): Call get_start and get_finish
on the endpoints to avoid storing packed ranges or ad-hoc
ranges in them.
(selftest::test_make_location_nonpure_range_endpoints): New function.
(selftest::input_c_tests): Call it.
* input.h (get_start): New inline function.

From-SVN: r239831

8 years agoPR c++/77379 - ABI tag on thunk
Jason Merrill [Mon, 29 Aug 2016 16:42:57 +0000 (12:42 -0400)]
PR c++/77379 - ABI tag on thunk

* mangle.c (maybe_check_abi_tags): Add version parm, handle thunks.
(mangle_thunk): Add thunk parameter.
* method.c (finish_thunk): Pass it.
* cp-tree.h: Declare it.

From-SVN: r239830

8 years agoFix assert in build_va_arg, case 2b
Tom de Vries [Mon, 29 Aug 2016 16:42:16 +0000 (16:42 +0000)]
Fix assert in build_va_arg, case 2b

2016-08-29  Tom de Vries  <tom@codesourcery.com>

* c-common.c (build_va_arg): Fix type comparison assert.

From-SVN: r239829

8 years agoReplace error_at with assert in build_va_arg
Tom de Vries [Mon, 29 Aug 2016 16:41:42 +0000 (16:41 +0000)]
Replace error_at with assert in build_va_arg

2016-08-29  Tom de Vries  <tom@codesourcery.com>

* c-common.c (build_va_arg): Replace first argument type error
with assert.

From-SVN: r239828

8 years agoHandle errors in both args of va_arg
Tom de Vries [Mon, 29 Aug 2016 16:41:33 +0000 (16:41 +0000)]
Handle errors in both args of va_arg

2016-08-29  Tom de Vries  <tom@codesourcery.com>

PR c/77398
* c-common.c (build_va_arg): Add first argument error.  Build va_arg
with error_mark_node as va_list instead of with illegal va_list.

* gimplify.c (gimplify_va_arg_expr): Replace first argument type error
with assert.

* g++.dg/ext/va-arg1.C: Add error check for illegal first argument.

From-SVN: r239827

8 years ago* Makefile.in (gcov-iov.h): Add dummy recipe.
Eric Botcazou [Mon, 29 Aug 2016 14:43:29 +0000 (14:43 +0000)]
* Makefile.in (gcov-iov.h): Add dummy recipe.

From-SVN: r239823

8 years ago* gcc.dg/ipa/iinline-attr.c: XFAIL on Visium.
Eric Botcazou [Mon, 29 Aug 2016 14:25:45 +0000 (14:25 +0000)]
* gcc.dg/ipa/iinline-attr.c: XFAIL on Visium.

From-SVN: r239821

8 years ago* config/nvptx/nvptx.c: #include tree-vrp.h.
Nathan Sidwell [Mon, 29 Aug 2016 12:49:10 +0000 (12:49 +0000)]
* config/nvptx/nvptx.c: #include tree-vrp.h.

From-SVN: r239820

8 years agoFix, reorganize, and clarify comparisons of anonymous types/components.
Fritz Reese [Mon, 29 Aug 2016 12:24:25 +0000 (12:24 +0000)]
Fix, reorganize, and clarify comparisons of anonymous types/components.

2016-08-29  Fritz Reese  <fritzoreese@gmail.com>

Fix, reorganize, and clarify comparisons of anonymous types/components.

PR fortran/77327
* interface.c (is_anonymous_component, is_anonymous_dt): New functions.
* interface.c (compare_components, gfc_compare_derived_types): Use new
functions.

* gfortran.dg/dec_structure_13.f90: New testcase.

From-SVN: r239819

8 years agoPR fortran/77261 Run random_3 testcase only when real(10) is available.
Janne Blomqvist [Mon, 29 Aug 2016 08:29:16 +0000 (11:29 +0300)]
PR fortran/77261 Run random_3 testcase only when real(10) is available.

From-SVN: r239817

8 years agoDaily bump.
GCC Administrator [Mon, 29 Aug 2016 00:16:15 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239814

8 years agoi386.c (ix86_option_override_internal): Re-sort a couple of processor_alias_table...
Uros Bizjak [Sun, 28 Aug 2016 16:22:22 +0000 (18:22 +0200)]
i386.c (ix86_option_override_internal): Re-sort a couple of processor_alias_table entries.

* config/i386/i386.c (ix86_option_override_internal): Re-sort
  a couple of processor_alias_table entries.

From-SVN: r239809

8 years agore PR target/77324 (ICE in push_reload, at reload.c:1350)
Eric Botcazou [Sun, 28 Aug 2016 08:50:09 +0000 (08:50 +0000)]
re PR target/77324 (ICE in push_reload, at reload.c:1350)

PR target/77324
* config/sparc/sparc.c (sparc_legitimate_address_p): Accept special
HIGH+LO construct during reload.

From-SVN: r239807

8 years agoTag {ms,sysv}_va_list_type_node with '{ms,sysv}_abi va_list' attribute
Tom de Vries [Sun, 28 Aug 2016 05:05:15 +0000 (05:05 +0000)]
Tag {ms,sysv}_va_list_type_node with '{ms,sysv}_abi va_list' attribute

2016-08-28  Tom de Vries  <tom@codesourcery.com>

PR lto/70955
* config/i386/i386.c (ix86_build_builtin_va_list_64): Tag type with
'sysv_abi va_list' attribute.
(ix86_build_builtin_va_list): Tag type with 'ms_abi va_list' attribute.
(ix86_canonical_va_list_type): Handle 'sysv_abi/ms_abi va_list'
attributes.

* gcc.dg/pr70955.c: New test.
* gcc.dg/lto/pr70955_0.c: Same.
* gcc.dg/lto/pr70955_1.c: Same.

From-SVN: r239805

8 years agoDaily bump.
GCC Administrator [Sun, 28 Aug 2016 00:16:14 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239804

8 years agomake stack_slot_list a vec<rtx>
Trevor Saunders [Sun, 28 Aug 2016 00:04:04 +0000 (00:04 +0000)]
make stack_slot_list a vec<rtx>

gcc/ChangeLog:

2016-08-27  Trevor Saunders  <tbsaunde+gcc@tbsaunde.org>

* emit-rtl.h (struct rtl_data): Make stack_slot_list a vector.
* emit-rtl.c (unshare_all_rtl_1): Adjust.
(unshare_all_rtl_again): Likewise.
* function.c (assign_stack_local_1): Likewise.
(assign_stack_temp_for_type): Likewise.

From-SVN: r239801

8 years agomake forced labels a vec
Trevor Saunders [Sun, 28 Aug 2016 00:03:59 +0000 (00:03 +0000)]
make forced labels a vec

gcc/ChangeLog:

2016-08-27  Trevor Saunders  <tbsaunde+gcc@tbsaunde.org>

* cfgbuild.c (make_edges): Adjust.
* cfgrtl.c (can_delete_label_p): Likewise.
* dwarf2cfi.c (create_trace_edges): Likewise.
* except.c (sjlj_emit_dispatch_table): Likewise.
* function.h (struct expr_status): make x_forced_labels a vector.
* jump.c (rebuild_jump_labels_1): Adjust.
* reload1.c (set_initial_label_offsets): Likewise.
* stmt.c (force_label_rtx): Likewise.
(expand_label): Likewise.

From-SVN: r239800

8 years agohaifa-sched.c: make ready_list an auto_vec<rtx_insn *>
Trevor Saunders [Sun, 28 Aug 2016 00:03:52 +0000 (00:03 +0000)]
haifa-sched.c: make ready_list an auto_vec<rtx_insn *>

gcc/ChangeLog:

2016-08-27  Trevor Saunders  <tbsaunde+gcc@tbsaunde.org>

* haifa-sched.c (fix_recovery_deps): Make ready_list a vector.

From-SVN: r239799

8 years agoFix folding of VECTOR_CST comparisons
Patrick Palka [Sat, 27 Aug 2016 22:00:17 +0000 (22:00 +0000)]
Fix folding of VECTOR_CST comparisons

gcc/ChangeLog:

PR tree-optimization/71077
PR tree-optimization/68542
* fold-const.c (fold_relational_const): Fix folding of
VECTOR_CST comparisons that have a scalar boolean result type.
(selftest::test_vector_folding): New static function.
(selftest::fold_const_c_tests): Call it.

gcc/testsuite/ChangeLog:

PR tree-optimization/71077
* gcc.target/i386/pr71077.c: New test.

From-SVN: r239798

8 years agoextend.texi (SPU Built-in Functions): Remove stale references to material formerly...
Gerald Pfeifer [Sat, 27 Aug 2016 18:15:59 +0000 (18:15 +0000)]
extend.texi (SPU Built-in Functions): Remove stale references to material formerly at IBM and Sony.

* doc/extend.texi (SPU Built-in Functions): Remove stale
references to material formerly at IBM and Sony.

From-SVN: r239797

8 years agore PR fortran/77380 (ICE in gfc_check_dependency, at fortran/dependency.c:1255)
Steven G. Kargl [Sat, 27 Aug 2016 15:45:35 +0000 (15:45 +0000)]
re PR fortran/77380 (ICE in gfc_check_dependency, at fortran/dependency.c:1255)

2016-08-27  Steven G. Kargl <kargl@gcc.gnu.org>

PR fortran/77380
* dependency.c (gfc_check_dependency): Do not assert with
-fcoarray=lib.

2016-08-27  Steven G. Kargl <kargl@gcc.gnu.org>

PR fortran/77380
* gfortran.dg/pr77380.f90: New test.

From-SVN: r239796

8 years agore PR fortran/77372 (ICE in simplify_ieee_selected_real_kind, at fortran/simplify...
Steven G. Kargl [Sat, 27 Aug 2016 15:38:55 +0000 (15:38 +0000)]
re PR fortran/77372 (ICE in simplify_ieee_selected_real_kind, at fortran/simplify.c:7049)

2016-08-27  Steven G. Kargl <kargl@gcc.gnu.org>

PR fortran/77372
simplify.c (simplify_ieee_selected_real_kind): Check for NULL pointers.

2016-08-27  Steven G. Kargl <kargl@gcc.gnu.org>

PR fortran/77372
gfortran.dg/pr77372.f90: New test.

From-SVN: r239795

8 years agore PR libstdc++/77356 (regex error for a ECMAScript syntax string)
Tim Shen [Sat, 27 Aug 2016 02:03:23 +0000 (02:03 +0000)]
re PR libstdc++/77356 (regex error for a ECMAScript syntax string)

PR libstdc++/77356
* include/bits/regex_compiler.tcc(_M_insert_bracket_matcher,
_M_expression_term): Modify to support dash literal.
* include/bits/regex_scanner.h: Add dash as a token type to make
a different from the mandated dash literal by escaping.
* include/bits/regex_scanner.tcc(_M_scan_in_bracket): Emit dash
token in bracket expression parsing.
* testsuite/28_regex/regression.cc: Add new testcases.

From-SVN: r239794

8 years agoDaily bump.
GCC Administrator [Sat, 27 Aug 2016 00:16:16 +0000 (00:16 +0000)]
Daily bump.

From-SVN: r239793

8 years agore PR target/77349 (AIX DWARF debugging offset in 64 bit mode)
David Edelsohn [Fri, 26 Aug 2016 23:51:27 +0000 (23:51 +0000)]
re PR target/77349 (AIX DWARF debugging offset in 64 bit mode)

PR target/77349
* config/rs6000/xcoff.h (DWARF_OFFSET_SIZE): Define as PTR_SIZE.

From-SVN: r239790

8 years agoAdd validation and consolidation of fix-it hints
David Malcolm [Fri, 26 Aug 2016 21:25:41 +0000 (21:25 +0000)]
Add validation and consolidation of fix-it hints

The first aspect of this patch is to add some checking of fix-it hints.
The idea is to put this checking within the rich_location machinery,
rather than requiring every diagnostic to implement it for itself.

The fixits within a rich_location are "atomic": all must be valid for
any to be applicable.

We reject any fixits involving locations above
LINE_MAP_MAX_LOCATION_WITH_COLS.

There's no guarantee that it's sane to modify a macro, so we reject
any fix-its that touch them.

For example, note the attempt to provide a fix-it for the definition
of the macro FIELD:

spellcheck-fields-2.c: In function ‘test_macro’:
spellcheck-fields-2.c:26:15: error: ‘union u’ has no member named ‘colour’; did you mean ‘color’?
 #define FIELD colour
               ^
               color
spellcheck-fields-2.c:27:15: note: in expansion of macro ‘FIELD’
   return ptr->FIELD;
               ^~~~~

After this patch, the fixit is not displayed:

spellcheck-fields-2.c: In function ‘test_macro’:
spellcheck-fields-2.c:26:15: error: ‘union u’ has no member named ‘colour’; did you mean ‘color’?
 #define FIELD colour
               ^
spellcheck-fields-2.c:27:15: note: in expansion of macro ‘FIELD’
   return ptr->FIELD;
               ^~~~~

We might want some way for a diagnostic to opt-in to fix-its that
affect macros, but for now it's simplest to reject them.

The other aspect of this patch is fix-it consolidation: in some cases
neighboring fix-its can be merged.  For example, in a diagnostic to
modernize old-style struct initializers from:

 struct s example = {
- foo: 1,
+ .foo = 1,
 };

one approach would be to replace the "foo" with ".foo" and the ":"
with " =".  This would give two "replace" fix-its:

  foo: 1,
  --- FIXIT 1
  .foo
     - FIXIT 2
     =

This patch allows them to be consolidated into a single "replace" fix-it:

  foo: 1,
  ----
  .foo =

gcc/ChangeLog:
* diagnostic-show-locus.c
(selftest::test_fixit_consolidation): New function.
(selftest::diagnostic_show_locus_c_tests): Call it.
* gcc-rich-location.h (gcc_rich_location): Eliminate unused
constructor based on source_range.

gcc/testsuite/ChangeLog:
* gcc.dg/spellcheck-fields-2.c (test): Move
dg-begin/end-multiline-output within function body.
(test_macro): New function.

libcpp/ChangeLog:
* include/line-map.h (rich_location): Eliminate unimplemented
constructor based on source_range.
(rich_location::get_last_fixit_hint): New method.
(rich_location::reject_impossible_fixit): New method.
(rich_location): Add fields m_line_table and
m_seen_impossible_fixit.
(fixit_hint::maybe_append_replace): New pure virtual function.
(fixit_insert::maybe_append_replace): New function.
(fixit_replace::maybe_append_replace): New function.
* line-map.c (rich_location::rich_location): Initialize
m_line_table and m_seen_impossible_fixit.
(rich_location::add_fixit_insert): Call
reject_impossible_fixit and bail out if true.
(column_before_p): New function.
(rich_location::add_fixit_replace): Call reject_impossible_fixit
and bail out if true.  Attempt to consolidate with neighboring
fixits.
(rich_location::get_last_fixit_hint): New method.
(rich_location::reject_impossible_fixit): New method.
(fixit_insert::maybe_append_replace): New method.
(fixit_replace::maybe_append_replace): New method.

From-SVN: r239789