{"id":808496,"url":"http://patchwork.ozlabs.org/api/1.2/patches/808496/?format=json","web_url":"http://patchwork.ozlabs.org/project/gcc/patch/20170901011500.15345-5-aoliva@redhat.com/","project":{"id":17,"url":"http://patchwork.ozlabs.org/api/1.2/projects/17/?format=json","name":"GNU Compiler Collection","link_name":"gcc","list_id":"gcc-patches.gcc.gnu.org","list_email":"gcc-patches@gcc.gnu.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170901011500.15345-5-aoliva@redhat.com>","list_archive_url":null,"date":"2017-09-01T01:14:56","name":"[5/9,SFN] Introduce -gstatement-frontiers option, enable debug markers","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"3ae1d4afee8615e1a08c3011ef58838dea1584b1","submitter":{"id":2058,"url":"http://patchwork.ozlabs.org/api/1.2/people/2058/?format=json","name":"Alexandre Oliva","email":"aoliva@redhat.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/gcc/patch/20170901011500.15345-5-aoliva@redhat.com/mbox/","series":[{"id":934,"url":"http://patchwork.ozlabs.org/api/1.2/series/934/?format=json","web_url":"http://patchwork.ozlabs.org/project/gcc/list/?series=934","date":"2017-09-01T01:14:52","name":"[1/9,SFN] adjust RTL insn-walking API","version":1,"mbox":"http://patchwork.ozlabs.org/series/934/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/808496/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/808496/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","mailing list gcc-patches@gcc.gnu.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=gcc.gnu.org\n\t(client-ip=209.132.180.131; helo=sourceware.org;\n\tenvelope-from=gcc-patches-return-461240-incoming=patchwork.ozlabs.org@gcc.gnu.org;\n\treceiver=)","ozlabs.org; dkim=pass (1024-bit key;\n\tunprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org\n\theader.b=\"VPnTASoD\"; dkim-atps=neutral","sourceware.org; auth=none","ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=aoliva@redhat.com"],"Received":["from sourceware.org (server1.sourceware.org [209.132.180.131])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xk1Xg43whz9sMN\n\tfor ;\n\tFri, 1 Sep 2017 11:16:19 +1000 (AEST)","(qmail 94160 invoked by alias); 1 Sep 2017 01:15:55 -0000","(qmail 94072 invoked by uid 89); 1 Sep 2017 01:15:55 -0000","from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by\n\tsourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP;\n\tFri, 01 Sep 2017 01:15:53 +0000","from smtp.corp.redhat.com\n\t(int-mx02.intmail.prod.int.phx2.redhat.com\n\t[10.5.11.12])\t(using TLSv1.2 with cipher AECDH-AES256-SHA\n\t(256/256 bits))\t(No client certificate requested)\tby\n\tmx1.redhat.com (Postfix) with ESMTPS id 566D65F723;\n\tFri, 1 Sep 2017 01:15:52 +0000 (UTC)","from freie.home (ovpn04.gateway.prod.ext.phx2.redhat.com\n\t[10.5.9.4])\tby smtp.corp.redhat.com (Postfix) with ESMTPS id\n\t10A4060F82; Fri, 1 Sep 2017 01:15:51 +0000 (UTC)","from frit.home (frit.home [172.31.160.7])\tby freie.home\n\t(8.15.2/8.15.2) with ESMTP id v811FZOi009211;\n\tThu, 31 Aug 2017 22:15:42 -0300"],"DomainKey-Signature":"a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id\n\t:list-unsubscribe:list-archive:list-post:list-help:sender:from\n\t:to:cc:subject:date:message-id:in-reply-to:references; q=dns; s=\n\tdefault; b=umIGPQjHmxGNvdwdFVF7CuEzbxDXeP6qCbz17dqDbz1aQ6yApqczj\n\t1Uz7Y7hyXGExk0+4KK/U76FWAQrKRRtoMfQxI8OKbwgp/9PoyG6VNevUx9OYeo8g\n\t7PN4XgHSgYaFno1UiHZUb9x1EG0vS1SVljIn3qYUH7ltENwi6XextQ=","DKIM-Signature":"v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id\n\t:list-unsubscribe:list-archive:list-post:list-help:sender:from\n\t:to:cc:subject:date:message-id:in-reply-to:references; s=\n\tdefault; bh=rmVv84OuejMJhZdOxthIX+lY4PU=; b=VPnTASoDF+zFEGV5RHa2\n\tuTjaxdFRWopRlWaWukHzbeEmq2Pft3BLrUQE3DyEo3bZOFfHu7hQ5h7MgjKaPf9L\n\t45xo22ZeY5L92pHO1YKTHv+v+6uYIy9jfHlKTUxOleMRWCW7ZYziR5KdAqZR6dsU\n\t4SXusmdFqfulG83tfPsYvAA=","Mailing-List":"contact gcc-patches-help@gcc.gnu.org; run by ezmlm","Precedence":"bulk","List-Id":"","List-Unsubscribe":"","List-Archive":"","List-Post":"","List-Help":"","Sender":"gcc-patches-owner@gcc.gnu.org","X-Virus-Found":"No","X-Spam-SWARE-Status":"No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0,\n\tGIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RP_MATCHES_RCVD,\n\tSPF_HELO_PASS autolearn=ham version=3.3.2 spammy=dots,\n\tprogressive, Activate, frontiers","X-HELO":"mx1.redhat.com","DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 566D65F723","From":"Alexandre Oliva ","To":"Richard Biener ","Cc":"GCC Patches ,\n\tAlexandre Oliva ","Subject":"[PATCH 5/9] [SFN] Introduce -gstatement-frontiers option,\n\tenable debug markers","Date":"Thu, 31 Aug 2017 22:14:56 -0300","Message-Id":"<20170901011500.15345-5-aoliva@redhat.com>","In-Reply-To":"","References":""},"content":"Introduce a command line option to enable statement frontiers, enabled\nby default in optimized builds with DWARF2+ debug information.\n\nThis patch depends on an earlier patch that completed the\ninfrastructure for debug markers, and on another patch that turns -g\ninto a negatable option prefix.\n\ngcc/ChangeLog\n\n\t* common.opt (gstatement-frontiers): New, setting\n\tdebug_nonbind_markers_p.\n\t* rtl.h (MAY_HAVE_DEBUG_MARKER_INSNS): Activate.\n\t* toplev.c (process_options): Autodetect value for debug statement\n\tfrontiers option.\n\t* tree.h (MAY_HAVE_DEBUG_MARKER_STMTS): Activate.\n\t* doc/invoke.texi (gstatement-frontiers, gno-statement-frontiers): New.\n---\n gcc/common.opt | 4 ++++\n gcc/doc/invoke.texi | 12 ++++++++++++\n gcc/rtl.h | 2 +-\n gcc/toplev.c | 4 ++++\n gcc/tree.h | 2 +-\n 5 files changed, 22 insertions(+), 2 deletions(-)","diff":"diff --git a/gcc/common.opt b/gcc/common.opt\nindex d18559a..b0748a2 100644\n--- a/gcc/common.opt\n+++ b/gcc/common.opt\n@@ -2874,6 +2874,10 @@ gstabs+\n Common Driver JoinedOrMissing Negative(gvms)\n Generate debug information in extended STABS format.\n \n+gstatement-frontiers\n+Common Driver Var(debug_nonbind_markers_p) Init(2)\n+Emit progressive recommended breakpoint locations.\n+\n gstrict-dwarf\n Common Driver Report Var(dwarf_strict) Init(0)\n Don't emit DWARF additions beyond selected version.\ndiff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi\nindex 6dbc362..082f122 100644\n--- a/gcc/doc/invoke.texi\n+++ b/gcc/doc/invoke.texi\n@@ -344,6 +344,7 @@ Objective-C and Objective-C++ Dialects}.\n -ggdb -grecord-gcc-switches -gno-record-gcc-switches @gol\n -gstabs -gstabs+ -gstrict-dwarf -gno-strict-dwarf @gol\n -gcolumn-info -gno-column-info @gol\n+-gstatement-frontiers -gno-statement-frontiers @gol\n -gvms -gxcoff -gxcoff+ -gz@r{[}=@var{type}@r{]} @gol\n -fdebug-prefix-map=@var{old}=@var{new} -fdebug-types-section @gol\n -feliminate-dwarf2-dups -fno-eliminate-unused-debug-types @gol\n@@ -6988,6 +6989,17 @@ Emit location column information into DWARF debugging information, rather\n than just file and line.\n This option is disabled by default.\n \n+@item -gstatement-frontiers\n+@item -gno-statement-frontiers\n+@opindex gstatement-frontiers\n+@opindex gno-statement-frontiers\n+This option causes GCC to create markers in the internal representation\n+at the beginning of statements, and to keep them roughly in place\n+throughout compilation, using them to guide the output of @code{is_stmt}\n+markers in the line number table. This is enabled by default when\n+compiling with optimization (@option{-Os}, @option{-O}, @option{-O2},\n+@dots{}), and outputting DWARF 2 debug information at the normal level.\n+\n @item -gz@r{[}=@var{type}@r{]}\n @opindex gz\n Produce compressed debug sections in DWARF format, if that is supported.\ndiff --git a/gcc/rtl.h b/gcc/rtl.h\nindex f7aa5fb..6ca6ecf 100644\n--- a/gcc/rtl.h\n+++ b/gcc/rtl.h\n@@ -815,7 +815,7 @@ struct GTY(()) rtvec_def {\n #define NONDEBUG_INSN_P(X) (INSN_P (X) && !DEBUG_INSN_P (X))\n \n /* Nonzero if DEBUG_MARKER_INSN_P may possibly hold. */\n-#define MAY_HAVE_DEBUG_MARKER_INSNS 0 /* debug_nonbind_markers_p */\n+#define MAY_HAVE_DEBUG_MARKER_INSNS debug_nonbind_markers_p\n /* Nonzero if DEBUG_BIND_INSN_P may possibly hold. */\n #define MAY_HAVE_DEBUG_BIND_INSNS flag_var_tracking_assignments\n /* Nonzero if DEBUG_INSN_P may possibly hold. */\ndiff --git a/gcc/toplev.c b/gcc/toplev.c\nindex d23714c..dea6f82 100644\n--- a/gcc/toplev.c\n+++ b/gcc/toplev.c\n@@ -1534,6 +1534,10 @@ process_options (void)\n warning_at (UNKNOWN_LOCATION, 0,\n \t\t\"var-tracking-assignments changes selective scheduling\");\n \n+ if (debug_nonbind_markers_p == AUTODETECT_VALUE)\n+ debug_nonbind_markers_p = optimize && debug_info_level >= DINFO_LEVEL_NORMAL\n+ && (write_symbols == DWARF2_DEBUG || write_symbols == VMS_AND_DWARF2_DEBUG);\n+\n if (flag_tree_cselim == AUTODETECT_VALUE)\n {\n if (HAVE_conditional_move)\ndiff --git a/gcc/tree.h b/gcc/tree.h\nindex 379efa8..bd4a5bc 100644\n--- a/gcc/tree.h\n+++ b/gcc/tree.h\n@@ -1127,7 +1127,7 @@ extern void omp_clause_range_check_failed (const_tree, const char *, int,\n ((int)TREE_INT_CST_LOW (VL_EXP_CHECK (NODE)->exp.operands[0]))\n \n /* Nonzero if gimple_debug_nonbind_marker_p() may possibly hold. */\n-#define MAY_HAVE_DEBUG_MARKER_STMTS 0 /* debug_nonbind_markers_p */\n+#define MAY_HAVE_DEBUG_MARKER_STMTS debug_nonbind_markers_p\n /* Nonzero if gimple_debug_bind_p() (and thus\n gimple_debug_source_bind_p()) may possibly hold. */\n #define MAY_HAVE_DEBUG_BIND_STMTS flag_var_tracking_assignments\n","prefixes":["5/9","SFN"]}