From patchwork Mon Oct 29 18:32:37 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aldy Hernandez X-Patchwork-Id: 195107 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id D0FC32C00D6 for ; Tue, 30 Oct 2012 05:32:50 +1100 (EST) Comment: DKIM? See http://www.dkim.org DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=gcc.gnu.org; s=default; x=1352140371; h=Comment: DomainKey-Signature:Received:Received:Received:Received:Received: Message-ID:Date:From:User-Agent:MIME-Version:To:CC:Subject: Content-Type:Mailing-List:Precedence:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:Sender:Delivered-To; bh=vIU2APi ZSIzhdqqAc+s5wtPosGU=; b=wPQLVa9WRU5UIsmj5op/pfOakIUrgE/hld9GJqX B1NFMbBdVzDtKMJPFqvc35bui5vRni1Tr/C6fxoa5A96GR0XOyBWBQ+xVgzd932y //UgkkDJA/B+/68G+mWN2WMNxxdKDxhGm3S7rRxsAJLMHV3wtFLXhaAlvN4eIYoY LOUs= Comment: DomainKeys? See http://antispam.yahoo.com/domainkeys DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=gcc.gnu.org; h=Received:Received:X-SWARE-Spam-Status:X-Spam-Check-By:Received:Received:Received:Message-ID:Date:From:User-Agent:MIME-Version:To:CC:Subject:Content-Type:Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:Sender:Delivered-To; b=ubwL7sQdE5pUriL/ug5FVU88gssewPoFIt4MOVVOjF//u3jq5CeN2lfPWKNjb/ EN89xV17U9HRAutJPd7qUPCMhBNMeh2ZotrWTmoLVNKH3yaEuEc2A8f4UGjx1LNT Z8QsFv0IY0wVWWZi+vkIECwg43YG2YW9oCoqecvODncsQ=; Received: (qmail 13805 invoked by alias); 29 Oct 2012 18:32:46 -0000 Received: (qmail 13795 invoked by uid 22791); 29 Oct 2012 18:32:45 -0000 X-SWARE-Spam-Status: No, hits=-6.6 required=5.0 tests=AWL, BAYES_00, KHOP_RCVD_UNTRUST, RCVD_IN_DNSWL_HI, RCVD_IN_HOSTKARMA_W, RP_MATCHES_RCVD, SPF_HELO_PASS, TW_TM X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 29 Oct 2012 18:32:39 +0000 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q9TIWcqV017431 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 29 Oct 2012 14:32:39 -0400 Received: from houston.quesejoda.com (vpn-8-145.rdu.redhat.com [10.11.8.145]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id q9TIWbCn000533; Mon, 29 Oct 2012 14:32:38 -0400 Message-ID: <508ECBC5.1020008@redhat.com> Date: Mon, 29 Oct 2012 13:32:37 -0500 From: Aldy Hernandez User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 MIME-Version: 1.0 To: gcc-patches , Richard Henderson CC: Torvald Riegel Subject: [patch] move GIMPLE_TRANSACTION expansion to tmmark pass 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 Delivered-To: mailing list gcc-patches@gcc.gnu.org What was supposed to be a simple fix, turned out to be a nightmare... I am working on adding an uninstrumented code path for TM, in expectation of hardware TM. This means that we eventually want to generate something like this: prop = _ITM_beginTransaction(PR_multiwayCode); if (prop & PR_UNINSTRUMENTEDCODE) global = 55; else _ITM_WU4 (&global, 55); _ITM_commitTransaction (); Since reading "prop" and duplicating BB's must happen at the TMMARK pass (instrumentation pass), we must move the expansion of GIMPLE_TRANSACTION -> _ITM_beginTransaction() into the TMMARK pass, instead of a later pass like is currently being done. Moving this, caused all sorts of grief, because I found that the abnormal edges for each TM built-in, have been incorrectly generated since day one. I have no idea, how things worked before... Every TM built-in must have an abnormal edge back to the outer-most transaction. As it currently stands, TM built-ins have an edge to the current transaction, which means that for nested transactions, we end up with incorrect edges to every transaction. Transaction cancel statements, OTOH, must _also_ have an edge to the inner transaction-- the transaction to which the cancel appears in. And finally, __transaction_cancel[[outer]] statements must only have one abnormal edge to the transaction marked as outer by the front-end (__transaction_atomic[[outer]]). I had to fix all this, as part of the work moving the transaction expansion into the TMMARK pass. There are also various cleanups that seemed sensible while I was re-arranging things, so my apologies for the plethora of fixes. I have not added further tests, since the present testsuite triggers the problems I found. Finally, I adapted various tests, because things now happen in other passes. OK, pending one more round of testing? p.s. You'll hate tm_region_for_stmt*(), but (a) it's only called once for every __transaction_cancel, which is rare (b) I was really tired :)). If you hate it more than I do, I can optimize it later. + * trans-mem.c (tm_log_emit_save_or_restores): Remove FALLTHRU_EDGE + parameter and calculate locally. + (struct tm_region): Add new fields: + original_transaction_was_outer, transaction_label, tm_state. + (tm_region_init_0): Initialize new tm_region fields. + (expand_assign_tm): Add comments. + (expand_transaction): Reorganize to be called in tmmark pass + instead of the tmedge pass. + (generate_tm_state): New. + (execute_tm_mark): Generate TM_STATE temporaries. Expand gimple + transactions into calls into the run-time. + (tm_region_for_stmt_1): New. + (tm_region_for_stmt): New. + (make_tm_edge): Point abnormal TM built-in edges to the outermost + transaction. Point abnormal TM_ABORT edges appropriately. + (tmedge_wire_transaction_restarts): Abstract code previously in + expand_transaction to wire abnormal edges. + (expand_regions_1): Rewrite to use callback. + (expand_regions): Same. + (execute_tm_edges): Call expand_regions with new callback. + * trans-mem.h (PR_MULTIWAYCODE): New. + * tree-cfg.c (gimple_debug_cfg): Update comment. + + testsuite/ + * gcc.dg/tm/debug-1.c: Scan for _ITM_beginTransaction. + * gcc.dg/tm/irrevocable-3.c: Scan for doesGoIrrevocable. + * gcc.dg/tm/irrevocable-4.c: Scan for hasNoIrrevocable. + * gcc.dg/tm/props-4.c: Do not scan for LABEL. + * gcc.dg/tm/memopt-11.c: Adjust scan text. + * gcc.dg/tm/memopt-10.c: Adjust scan text. diff --git a/gcc/testsuite/gcc.dg/tm/debug-1.c b/gcc/testsuite/gcc.dg/tm/debug-1.c index fae5d6b..01acfae 100644 --- a/gcc/testsuite/gcc.dg/tm/debug-1.c +++ b/gcc/testsuite/gcc.dg/tm/debug-1.c @@ -20,7 +20,7 @@ main() { } /* { dg-final { scan-tree-dump-times ": 13:.*b = 9898" 1 "tmmark" } } */ -/* { dg-final { scan-tree-dump-times ": 14:.*__transaction" 1 "tmmark" } } */ +/* { dg-final { scan-tree-dump-times ": 14:.*_ITM_beginTransaction" 1 "tmmark" } } */ /* { dg-final { scan-tree-dump-times ": 15:.*ITM_WU. \\(&z" 1 "tmmark" } } */ /* { dg-final { scan-tree-dump-times ": 16:.*ITM_WU. \\(&a" 1 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/testsuite/gcc.dg/tm/irrevocable-3.c b/gcc/testsuite/gcc.dg/tm/irrevocable-3.c index c085479..fdf3e52 100644 --- a/gcc/testsuite/gcc.dg/tm/irrevocable-3.c +++ b/gcc/testsuite/gcc.dg/tm/irrevocable-3.c @@ -10,5 +10,5 @@ foo() } } -/* { dg-final { scan-tree-dump-times "GTMA_MAY_ENTER_IRREVOCABLE" 1 "tmmark" } } */ +/* { dg-final { scan-tree-dump-times "doesGoIrrevocable" 1 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/testsuite/gcc.dg/tm/irrevocable-4.c b/gcc/testsuite/gcc.dg/tm/irrevocable-4.c index ee759b8..72075df 100644 --- a/gcc/testsuite/gcc.dg/tm/irrevocable-4.c +++ b/gcc/testsuite/gcc.dg/tm/irrevocable-4.c @@ -12,5 +12,5 @@ foo() } } -/* { dg-final { scan-tree-dump-times "GTMA_MAY_ENTER_IRREVOCABLE" 1 "tmmark" } } */ +/* { dg-final { scan-tree-dump-times "hasNoIrrevocable" 0 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/testsuite/gcc.dg/tm/memopt-10.c b/gcc/testsuite/gcc.dg/tm/memopt-10.c index 5caa6b5..0978bce 100644 --- a/gcc/testsuite/gcc.dg/tm/memopt-10.c +++ b/gcc/testsuite/gcc.dg/tm/memopt-10.c @@ -24,5 +24,5 @@ int f() /* { dg-final { scan-tree-dump-times "ITM_LU" 0 "tmmark" } } */ /* { dg-final { scan-tree-dump-times "ITM_WU" 0 "tmmark" } } */ -/* { dg-final { scan-tree-dump-times "tm_save" 1 "tmmark" } } */ +/* { dg-final { scan-tree-dump-times "int tm_save" 1 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/testsuite/gcc.dg/tm/memopt-11.c b/gcc/testsuite/gcc.dg/tm/memopt-11.c index 07972a4..36aa664 100644 --- a/gcc/testsuite/gcc.dg/tm/memopt-11.c +++ b/gcc/testsuite/gcc.dg/tm/memopt-11.c @@ -25,5 +25,5 @@ int f() /* { dg-final { scan-tree-dump-times "ITM_LU" 0 "tmmark" } } */ /* { dg-final { scan-tree-dump-times "ITM_WU" 0 "tmmark" } } */ -/* { dg-final { scan-tree-dump-times "tm_save" 1 "tmmark" } } */ +/* { dg-final { scan-tree-dump-times "int tm_save" 1 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/testsuite/gcc.dg/tm/props-4.c b/gcc/testsuite/gcc.dg/tm/props-4.c index c9d0c2b..c34f5e6 100644 --- a/gcc/testsuite/gcc.dg/tm/props-4.c +++ b/gcc/testsuite/gcc.dg/tm/props-4.c @@ -22,6 +22,5 @@ foo(void) /* { dg-final { scan-tree-dump-times " instrumentedCode" 1 "tmedge" } } */ /* { dg-final { scan-tree-dump-times "hasNoAbort" 0 "tmedge" } } */ -/* { dg-final { scan-tree-dump-times "LABEL=" 1 "tmmark" } } */ /* { dg-final { cleanup-tree-dump "tmedge" } } */ /* { dg-final { cleanup-tree-dump "tmmark" } } */ diff --git a/gcc/trans-mem.c b/gcc/trans-mem.c index 211c45e..491f63c 100644 --- a/gcc/trans-mem.c +++ b/gcc/trans-mem.c @@ -133,6 +133,10 @@ over: */ +static void *expand_regions (struct tm_region *, + void *(*callback)(struct tm_region *, void *), + void *); + /* Return the attributes we want to examine for X, or NULL if it's not something we examine. We look at function types, but allow pointers @@ -1247,8 +1251,7 @@ tm_log_emit_restores (basic_block entry_block, basic_block bb) TRXN_PROP is either A_SAVELIVEVARIABLES or A_RESTORELIVEVARIABLES. The code sequence is inserted in a new basic block created in - END_BB which is inserted between BEFORE_BB and the destination of - FALLTHRU_EDGE. + END_BB which is inserted between BEFORE_BB and its fall thru edge. STATUS is the return value from _ITM_beginTransaction. ENTRY_BLOCK is the entry block for the transaction. @@ -1262,13 +1265,13 @@ tm_log_emit_save_or_restores (basic_block entry_block, tree status, void (*emitf)(basic_block, basic_block), basic_block before_bb, - edge fallthru_edge, basic_block *end_bb) { basic_block cond_bb, code_bb; gimple cond_stmt, stmt; gimple_stmt_iterator gsi; tree t1, t2; + edge fallthru_edge = FALLTHRU_EDGE (before_bb); int old_flags = fallthru_edge->flags; cond_bb = create_empty_bb (before_bb); @@ -1765,9 +1768,23 @@ struct tm_region /* Link to the next outer transaction. */ struct tm_region *outer; - /* The GIMPLE_TRANSACTION statement beginning this transaction. */ + /* The GIMPLE_TRANSACTION statement beginning this transaction. + After TM_MARK, this gets replaced by a call to + BUILT_IN_TM_START. */ gimple transaction_stmt; + /* After TM_MARK expands the GIMPLE_TRANSACTION into a call to + BUILT_IN_TM_START, this field is true if the transaction is an + outer transaction. */ + bool original_transaction_was_outer; + + /* This holds the transaction label from the original + GIMPLE_TRANSACTION statement. */ + tree transaction_label; + + /* Return value from BUILT_IN_TM_START. */ + tree tm_state; + /* The entry block to this region. */ basic_block entry_block; @@ -1819,6 +1836,9 @@ tm_region_init_0 (struct tm_region *outer, basic_block bb, gimple stmt) region->outer = outer; region->transaction_stmt = stmt; + region->original_transaction_was_outer = false; + region->transaction_label = gimple_transaction_label (stmt); + region->tm_state = NULL; /* There are either one or two edges out of the block containing the GIMPLE_TRANSACTION, one to the actual region and one to the @@ -2192,6 +2212,7 @@ expand_assign_tm (struct tm_region *region, gimple_stmt_iterator *gsi) return; } + // Remove original load/store statement. gsi_remove (gsi, true); if (load_p && !store_p) @@ -2245,7 +2266,9 @@ expand_assign_tm (struct tm_region *region, gimple_stmt_iterator *gsi) if (!store_p) requires_barrier (region->entry_block, lhs, gcall); - /* add_stmt_to_tm_region (region, gcall); */ + // The calls to build_tm_{store,load} above inserted the instrumented + // call into the stream. + // gsi_insert_before (gsi, gcall, GSI_SAME_STMT); } @@ -2515,6 +2538,128 @@ compute_transaction_bits (void) bitmap_obstack_release (&tm_obstack); } +/* Replace the GIMPLE_TRANSACTION in this region with the corresponding + call to BUILT_IN_TM_START. */ + +static void * +expand_transaction (struct tm_region *region, void *data ATTRIBUTE_UNUSED) +{ + int flags = PR_MULTIWAYCODE; + tree tm_start = builtin_decl_explicit (BUILT_IN_TM_START); + + /* ??? There are plenty of bits here we're not computing. */ + int subcode = gimple_transaction_subcode (region->transaction_stmt); + if (subcode & GTMA_DOES_GO_IRREVOCABLE) + flags |= PR_DOESGOIRREVOCABLE; + if ((subcode & GTMA_MAY_ENTER_IRREVOCABLE) == 0) + flags |= PR_HASNOIRREVOCABLE; + /* If the transaction does not have an abort in lexical scope and is not + marked as an outer transaction, then it will never abort. */ + if ((subcode & GTMA_HAVE_ABORT) == 0 + && (subcode & GTMA_IS_OUTER) == 0) + flags |= PR_HASNOABORT; + if ((subcode & GTMA_HAVE_STORE) == 0) + flags |= PR_READONLY; + if (subcode & GTMA_IS_OUTER) + region->original_transaction_was_outer = true; + tree t = build_int_cst (TREE_TYPE (region->tm_state), flags); + gimple call = gimple_build_call (tm_start, 1, t); + gimple_call_set_lhs (call, region->tm_state); + gimple_set_location (call, gimple_location (region->transaction_stmt)); + + basic_block transaction_bb = gimple_bb (region->transaction_stmt); + + // Generate log saves. + if (!VEC_empty (tree, tm_log_save_addresses)) + tm_log_emit_saves (region->entry_block, transaction_bb); + + // Replace the GIMPLE_TRANSACTION with the call to BUILT_IN_TM_START. + gimple_stmt_iterator gsi = gsi_last_bb (transaction_bb); + gcc_assert (gsi_stmt (gsi) == region->transaction_stmt); + gsi_insert_before (&gsi, call, GSI_SAME_STMT); + gsi_remove (&gsi, true); + + // Generate log restores. + basic_block slice_bb; + if (!VEC_empty (tree, tm_log_save_addresses)) + region->entry_block = + tm_log_emit_save_or_restores (region->entry_block, + A_RESTORELIVEVARIABLES, + region->tm_state, + tm_log_emit_restores, + transaction_bb, + &slice_bb); + else + slice_bb = transaction_bb; + + /* If we have an ABORT edge, create a test following the start + call to perform the abort. */ + if (region->transaction_label) + { + basic_block test_bb = create_empty_bb (slice_bb); + if (current_loops && slice_bb->loop_father) + add_bb_to_loop (test_bb, slice_bb->loop_father); + if (VEC_empty (tree, tm_log_save_addresses)) + region->entry_block = test_bb; + + tree t1 = create_tmp_reg (TREE_TYPE (region->tm_state), NULL); + tree t2 = build_int_cst (TREE_TYPE (region->tm_state), A_ABORTTRANSACTION); + gimple stmt = gimple_build_assign_with_ops (BIT_AND_EXPR, t1, + region->tm_state, t2); + gimple_stmt_iterator gsi = gsi_last_bb (test_bb); + gsi_insert_after (&gsi, stmt, GSI_CONTINUE_LINKING); + + t2 = build_int_cst (TREE_TYPE (region->tm_state), 0); + stmt = gimple_build_cond (NE_EXPR, t1, t2, NULL, NULL); + gsi_insert_after (&gsi, stmt, GSI_CONTINUE_LINKING); + + edge e = FALLTHRU_EDGE (slice_bb); + redirect_edge_pred (e, test_bb); + e->flags = EDGE_FALSE_VALUE; + e->probability = PROB_ALWAYS - PROB_VERY_UNLIKELY; + + e = BRANCH_EDGE (transaction_bb); + redirect_edge_pred (e, test_bb); + e->flags = EDGE_TRUE_VALUE; + e->probability = PROB_VERY_UNLIKELY; + + e = make_edge (slice_bb, test_bb, EDGE_FALLTHRU); + } + /* Otherwise, if we have no abort, but we have PHIs at the beginning + of the atomic region, this means we have a loop at the beginning + of the atomic region that shares the first block. This can cause + problems with the transaction restart abnormal edges to be added + in the tm_edges pass. Solve this by adding a new empty block to + receive the abnormal edges. */ + else if (phi_nodes (region->entry_block)) + { + basic_block empty_bb = + region->entry_block = create_empty_bb (transaction_bb); + if (current_loops && transaction_bb->loop_father) + add_bb_to_loop (empty_bb, transaction_bb->loop_father); + + edge e = FALLTHRU_EDGE (transaction_bb); + redirect_edge_pred (e, empty_bb); + + e = make_edge (transaction_bb, empty_bb, EDGE_FALLTHRU); + } + + region->transaction_stmt = call; + return NULL; +} + +/* Generate the temporary to be used for the return value of + BUILT_IN_TM_START. */ + +static void * +generate_tm_state (struct tm_region *region, void *data ATTRIBUTE_UNUSED) +{ + tree tm_start = builtin_decl_explicit (BUILT_IN_TM_START); + region->tm_state = + create_tmp_reg (TREE_TYPE (TREE_TYPE (tm_start)), "tm_state"); + return NULL; +} + /* Entry point to the MARK phase of TM expansion. Here we replace transactional memory statements with calls to builtins, and function calls with their transactional clones (if available). But we don't @@ -2531,9 +2676,12 @@ execute_tm_mark (void) queue = VEC_alloc (basic_block, heap, 10); pending_edge_inserts_p = false; + expand_regions (all_tm_regions, generate_tm_state, NULL); + for (region = all_tm_regions; region ; region = region->next) { tm_log_init (); + /* If we have a transaction... */ if (region->exit_blocks) { @@ -2561,8 +2709,12 @@ execute_tm_mark (void) tm_log_emit (); } + // Expand GIMPLE_TRANSACTIONs into calls into the runtime. + expand_regions (all_tm_regions, expand_transaction, NULL); + if (pending_edge_inserts_p) gsi_commit_edge_inserts (); + free_dominance_info (CDI_DOMINATORS); return 0; } @@ -2586,8 +2738,62 @@ struct gimple_opt_pass pass_tm_mark = } }; -/* Create an abnormal call edge from BB to the first block of the region - represented by STATE. Also record the edge in the TM_RESTART map. */ + +/* A helper function for tm_region_for_stmt. Search the instructions + in REGION and return the one that contains the gimple statement in + DATA. If the statement in DATA is not found, return NULL. */ + +static void * +tm_region_for_stmt_1 (struct tm_region *region, void *data) +{ + void *ret = NULL; + gimple stmt = (gimple) data; + unsigned int i; + basic_block bb; + VEC (basic_block, heap) *queue; + + queue = get_tm_region_blocks (region->entry_block, + region->exit_blocks, + region->irr_blocks, + NULL, + /*stop_at_irr_p=*/false); + for (i = 0; VEC_iterate (basic_block, queue, i, bb); ++i) + { + gimple_stmt_iterator gsi = gsi_start_bb (bb); + for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi)) + if (gsi_stmt (gsi) == stmt) + { + ret = (void *) region; + goto exit; + } + } + exit: + VEC_free (basic_block, heap, queue); + return ret; +} + +/* Find the region that contains STMT. This implementation is a + temporary measure (read inefficient). We should either keep this + information on the side, or find it through some clever EH landing + pad searches. + + ?? We should be able to get the inner transaction via the region_nr + saved on STMT, and read the transaction_stmt from that, and find + the first region block from there. */ + +static struct tm_region * +tm_region_for_stmt (gimple stmt) +{ + struct tm_region *region; + region = (struct tm_region *) expand_regions (all_tm_regions, + tm_region_for_stmt_1, + (void *) stmt); + return region; +} + +/* Create abnormal edges from the BB to the appropriate transaction(s) + depending on whether we have a TM_ABORT or another TM builtin. + Also record the edge in the TM_RESTART map. */ static inline void make_tm_edge (gimple stmt, basic_block bb, struct tm_region *region) @@ -2616,6 +2822,47 @@ make_tm_edge (gimple stmt, basic_block bb, struct tm_region *region) n->label_or_list = tree_cons (NULL, dummy.label_or_list, old); } + if (DECL_FUNCTION_CODE (gimple_call_fndecl (stmt)) == BUILT_IN_TM_ABORT) + { + // If we have a ``_transaction_cancel [[outer]]'', there is only + // one abnormal edge: to the transaction marked OUTER. + tree arg = gimple_call_arg (stmt, 0); + if (TREE_CODE (arg) == INTEGER_CST) + { + if (TREE_INT_CST_LOW (arg) & AR_OUTERABORT) + { + // Find the GTMA_IS_OUTER transaction. + for (; region; region = region->outer) + { + if (region->original_transaction_was_outer) + { + make_edge (bb, region->entry_block, EDGE_ABNORMAL); + return; + } + } + // The front-end should have caught outer aborts without + // an outer transaction. + gcc_unreachable (); + } + } + else + { + /* Some idiot called BUILT_IN_TM_ABORT directly and passed a + non-integer. Let's hope they really know what they're + doing, because we're about to set inner and outer edges for + them. */ + } + + // By default, TM aborts have an abnormal edge to the inner-most + // transaction. + struct tm_region *inner = tm_region_for_stmt (stmt); + if (inner) + make_edge (bb, inner->entry_block, EDGE_ABNORMAL); + } + + // All TM builtins have an abnormal edge to the outer-most transaction. + while (region->outer) + region = region->outer; make_edge (bb, region->entry_block, EDGE_ABNORMAL); } @@ -2633,13 +2880,6 @@ expand_block_edges (struct tm_region *region, basic_block bb) bool do_next = true; gimple stmt = gsi_stmt (gsi); - /* ??? TM_COMMIT (and any other tm builtin function) in a nested - transaction has an abnormal edge back to the outer-most transaction - (there are no nested retries), while a TM_ABORT also has an abnormal - backedge to the inner-most transaction. We haven't actually saved - the inner-most transaction here. We should be able to get to it - via the region_nr saved on STMT, and read the transaction_stmt from - that, and find the first region block from there. */ /* ??? Shouldn't we split for any non-pure, non-irrevocable function? */ if (gimple_code (stmt) == GIMPLE_CALL && (gimple_call_flags (stmt) & ECF_TM_BUILTIN) != 0) @@ -2666,163 +2906,76 @@ expand_block_edges (struct tm_region *region, basic_block bb) } } -/* Expand the GIMPLE_TRANSACTION statement into the STM library call. */ +/* Wire transaction restart edges and split BB's at each TM builtin + function. */ -static void -expand_transaction (struct tm_region *region) +static void * +tmedge_wire_transaction_restarts (struct tm_region *region, + void *data ATTRIBUTE_UNUSED) { - tree status, tm_start; - basic_block atomic_bb, slice_bb; - gimple_stmt_iterator gsi; - tree t1, t2; - gimple g; - int flags, subcode; - - tm_start = builtin_decl_explicit (BUILT_IN_TM_START); - status = create_tmp_reg (TREE_TYPE (TREE_TYPE (tm_start)), "tm_state"); - - /* ??? There are plenty of bits here we're not computing. */ - subcode = gimple_transaction_subcode (region->transaction_stmt); - if (subcode & GTMA_DOES_GO_IRREVOCABLE) - flags = PR_DOESGOIRREVOCABLE | PR_UNINSTRUMENTEDCODE; - else - flags = PR_INSTRUMENTEDCODE; - if ((subcode & GTMA_MAY_ENTER_IRREVOCABLE) == 0) - flags |= PR_HASNOIRREVOCABLE; - /* If the transaction does not have an abort in lexical scope and is not - marked as an outer transaction, then it will never abort. */ - if ((subcode & GTMA_HAVE_ABORT) == 0 - && (subcode & GTMA_IS_OUTER) == 0) - flags |= PR_HASNOABORT; - if ((subcode & GTMA_HAVE_STORE) == 0) - flags |= PR_READONLY; - t2 = build_int_cst (TREE_TYPE (status), flags); - g = gimple_build_call (tm_start, 1, t2); - gimple_call_set_lhs (g, status); - gimple_set_location (g, gimple_location (region->transaction_stmt)); - - atomic_bb = gimple_bb (region->transaction_stmt); - - if (!VEC_empty (tree, tm_log_save_addresses)) - tm_log_emit_saves (region->entry_block, atomic_bb); - - gsi = gsi_last_bb (atomic_bb); - gsi_insert_before (&gsi, g, GSI_SAME_STMT); - gsi_remove (&gsi, true); - - if (!VEC_empty (tree, tm_log_save_addresses)) - region->entry_block = - tm_log_emit_save_or_restores (region->entry_block, - A_RESTORELIVEVARIABLES, - status, - tm_log_emit_restores, - atomic_bb, - FALLTHRU_EDGE (atomic_bb), - &slice_bb); - else - slice_bb = atomic_bb; - - /* If we have an ABORT statement, create a test following the start - call to perform the abort. */ - if (gimple_transaction_label (region->transaction_stmt)) - { - edge e; - basic_block test_bb; - - test_bb = create_empty_bb (slice_bb); - if (current_loops && slice_bb->loop_father) - add_bb_to_loop (test_bb, slice_bb->loop_father); - if (VEC_empty (tree, tm_log_save_addresses)) - region->entry_block = test_bb; - gsi = gsi_last_bb (test_bb); - - t1 = create_tmp_reg (TREE_TYPE (status), NULL); - t2 = build_int_cst (TREE_TYPE (status), A_ABORTTRANSACTION); - g = gimple_build_assign_with_ops (BIT_AND_EXPR, t1, status, t2); - gsi_insert_after (&gsi, g, GSI_CONTINUE_LINKING); - - t2 = build_int_cst (TREE_TYPE (status), 0); - g = gimple_build_cond (NE_EXPR, t1, t2, NULL, NULL); - gsi_insert_after (&gsi, g, GSI_CONTINUE_LINKING); - - e = FALLTHRU_EDGE (slice_bb); - redirect_edge_pred (e, test_bb); - e->flags = EDGE_FALSE_VALUE; - e->probability = PROB_ALWAYS - PROB_VERY_UNLIKELY; - - e = BRANCH_EDGE (atomic_bb); - redirect_edge_pred (e, test_bb); - e->flags = EDGE_TRUE_VALUE; - e->probability = PROB_VERY_UNLIKELY; - - e = make_edge (slice_bb, test_bb, EDGE_FALLTHRU); - } - - /* If we've no abort, but we do have PHIs at the beginning of the atomic - region, that means we've a loop at the beginning of the atomic region - that shares the first block. This can cause problems with the abnormal - edges we're about to add for the transaction restart. Solve this by - adding a new empty block to receive the abnormal edges. */ - else if (phi_nodes (region->entry_block)) - { - edge e; - basic_block empty_bb; - - region->entry_block = empty_bb = create_empty_bb (atomic_bb); - if (current_loops && atomic_bb->loop_father) - add_bb_to_loop (empty_bb, atomic_bb->loop_father); - - e = FALLTHRU_EDGE (atomic_bb); - redirect_edge_pred (e, empty_bb); - - e = make_edge (atomic_bb, empty_bb, EDGE_FALLTHRU); - } + unsigned int i; + basic_block bb; + VEC (basic_block, heap) *queue; - /* The GIMPLE_TRANSACTION statement no longer exists. */ - region->transaction_stmt = NULL; + /* Collect the set of blocks in this region. Do this before + splitting edges, so that we don't have to play with the + dominator tree in the middle. */ + queue = get_tm_region_blocks (region->entry_block, + region->exit_blocks, + region->irr_blocks, + NULL, + /*stop_at_irr_p=*/false); + for (i = 0; VEC_iterate (basic_block, queue, i, bb); ++i) + expand_block_edges (region, bb); + VEC_free (basic_block, heap, queue); + return NULL; } -static void expand_regions (struct tm_region *); - /* Helper function for expand_regions. Expand REGION and recurse to - the inner region. */ + the inner region. Call CALLBACK on each region. CALLBACK returns + NULL to continue the traversal, otherwise a non-null value which + this function will return as well. */ -static void -expand_regions_1 (struct tm_region *region) +static void * +expand_regions_1 (struct tm_region *region, + void *(*callback)(struct tm_region *, void *), + void *data) { + void *retval = NULL; if (region->exit_blocks) { - unsigned int i; - basic_block bb; - VEC (basic_block, heap) *queue; - - /* Collect the set of blocks in this region. Do this before - splitting edges, so that we don't have to play with the - dominator tree in the middle. */ - queue = get_tm_region_blocks (region->entry_block, - region->exit_blocks, - region->irr_blocks, - NULL, - /*stop_at_irr_p=*/false); - expand_transaction (region); - for (i = 0; VEC_iterate (basic_block, queue, i, bb); ++i) - expand_block_edges (region, bb); - VEC_free (basic_block, heap, queue); + retval = callback (region, data); + if (retval) + return retval; } if (region->inner) - expand_regions (region->inner); + { + retval = expand_regions (region->inner, callback, data); + if (retval) + return retval; + } + return retval; } -/* Expand regions starting at REGION. */ +/* Traverse the regions enclosed and including REGION. Execute + CALLBACK for each region, passing DATA. CALLBACK returns NULL to + continue the traversal, otherwise a non-null value which this + function will return as well. */ -static void -expand_regions (struct tm_region *region) +static void * +expand_regions (struct tm_region *region, + void *(*callback)(struct tm_region *, void *), + void *data) { + void *retval = NULL; while (region) { - expand_regions_1 (region); + retval = expand_regions_1 (region, callback, data); + if (retval) + return retval; region = region->next; } + return retval; } /* Entry point to the final expansion of transactional nodes. */ @@ -2830,7 +2983,7 @@ expand_regions (struct tm_region *region) static unsigned int execute_tm_edges (void) { - expand_regions (all_tm_regions); + expand_regions (all_tm_regions, tmedge_wire_transaction_restarts, NULL); tm_log_delete (); /* We've got to release the dominance info now, to indicate that it diff --git a/gcc/trans-mem.h b/gcc/trans-mem.h index 95e9e7e..58ad2a2 100644 --- a/gcc/trans-mem.h +++ b/gcc/trans-mem.h @@ -21,6 +21,7 @@ /* These defines must match the enumerations in libitm.h. */ #define PR_INSTRUMENTEDCODE 0x0001 #define PR_UNINSTRUMENTEDCODE 0x0002 +#define PR_MULTIWAYCODE (PR_INSTRUMENTEDCODE | PR_UNINSTRUMENTEDCODE) #define PR_HASNOXMMUPDATE 0x0004 #define PR_HASNOABORT 0x0008 #define PR_HASNOIRREVOCABLE 0x0020 diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index 7fc5a53..c68d6eb 100644 --- a/gcc/tree-cfg.c +++ b/gcc/tree-cfg.c @@ -2061,7 +2061,7 @@ gimple_debug_bb_n (int n) /* Dump the CFG on stderr. FLAGS are the same used by the tree dumping functions - (see TDF_* in tree-pass.h). */ + (see TDF_* in dumpfile.h). */ void gimple_debug_cfg (int flags) @@ -6736,7 +6736,7 @@ move_sese_region_to_fn (struct function *dest_cfun, basic_block entry_bb, } -/* Dump FUNCTION_DECL FN to file FILE using FLAGS (see TDF_* in tree-pass.h) +/* Dump FUNCTION_DECL FN to file FILE using FLAGS (see TDF_* in dumpfile.h) */ void