From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20033 invoked by alias); 23 Dec 2016 19:59:12 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 20024 invoked by uid 89); 23 Dec 2016 19:59:11 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=0.7 required=5.0 tests=AWL,BAYES_50,KAM_STOCKGEN,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=no version=3.3.2 spammy=spots, TYPE_CODE, sk:dwarf2_, type_code X-HELO: gproxy1.mail.unifiedlayer.com Received: from gproxy1-pub.mail.unifiedlayer.com (HELO gproxy1.mail.unifiedlayer.com) (69.89.25.95) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 23 Dec 2016 19:59:01 +0000 Received: from cmgw4 (cmgw5 [10.0.90.85]) by gproxy1.mail.unifiedlayer.com (Postfix) with ESMTP id D68C4175E12 for ; Fri, 23 Dec 2016 12:58:59 -0700 (MST) Received: from box522.bluehost.com ([74.220.219.122]) by cmgw4 with id PXyw1u00J2f2jeq01XyzoQ; Fri, 23 Dec 2016 12:58:59 -0700 X-Authority-Analysis: v=2.1 cv=DKocvU9b c=1 sm=1 tr=0 a=GsOEXm/OWkKvwdLVJsfwcA==:117 a=GsOEXm/OWkKvwdLVJsfwcA==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=n5n_aSjo0skA:10 a=zstS-IiYAAAA:8 a=p-HpoobHz90TcP-_wrgA:9 a=9yGvTJM0hFUvHgym:21 a=eM6BKYUXkgZmmV1u:21 a=4G6NA9xxw8l3yy4pmD5M:22 Received: from 75-171-175-27.hlrn.qwest.net ([75.171.175.27]:45416 helo=bapiya) by box522.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.86_1) (envelope-from ) id 1cKVzU-00050w-5N; Fri, 23 Dec 2016 12:58:56 -0700 From: Tom Tromey To: Tom Tromey Cc: Pedro Alves , gdb-patches@sourceware.org Subject: Re: [RFA 6/8] Use value_freer in dwarf2_evaluate_loc_desc_full References: <1480395946-10924-1-git-send-email-tom@tromey.com> <1480395946-10924-7-git-send-email-tom@tromey.com> <06e2b7b1-feca-29f7-7e49-e05f01d05485@redhat.com> <87bmwgez57.fsf@tromey.com> <87wpeq5uvy.fsf@tromey.com> Date: Fri, 23 Dec 2016 19:59:00 -0000 In-Reply-To: <87wpeq5uvy.fsf@tromey.com> (Tom Tromey's message of "Fri, 23 Dec 2016 12:05:05 -0700") Message-ID: <87poki5se8.fsf@tromey.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.90 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-BWhitelist: no X-Exim-ID: 1cKVzU-00050w-5N X-Source-Sender: 75-171-175-27.hlrn.qwest.net (bapiya) [75.171.175.27]:45416 X-Source-Auth: tom+tromey.com X-Email-Count: 4 X-Source-Cap: ZWx5bnJvYmk7ZWx5bnJvYmk7Ym94NTIyLmJsdWVob3N0LmNvbQ== X-SW-Source: 2016-12/txt/msg00406.txt.bz2 >>>>> "Tom" == Tom Tromey writes: Tom> It looks good to me. I've made this change and I'll send the new Tom> patches soon. Here's the new patch to add scoped_value_mark. Tom commit 6f8663e26b0498790d3314f0649c5c9c923a48a0 Author: Tom Tromey Date: Mon Nov 21 18:02:11 2016 -0700 Add scoped_value_mark This adds a scoped_value_mark class, that records the value mark in the constructor and then calls value_free_to_mark in the destructor. It then updates various spots in gdb to use this class, rather than a cleanup. It would be better overall to replace "struct value *" with a shared_ptr, maybe eliminating the need for this class (watchpoints would perhaps need some new mechanism as well). However, that's difficult to do. 2016-12-13 Tom Tromey * python/py-value.c (valpy_dereference, valpy_referenced_value) (valpy_reference_value, valpy_const_value, valpy_get_address) (valpy_get_dynamic_type, valpy_lazy_string, valpy_do_cast) (valpy_getitem, valpy_call, valpy_binop_throw, valpy_negative) (valpy_absolute, valpy_richcompare_throw): Use scoped_value_mark. * dwarf2loc.c (dwarf2_loc_desc_get_symbol_read_needs): Use scoped_value_mark. * dwarf2-frame.c (execute_stack_op): Use scoped_value_mark. * value.h (scoped_value_mark): New class. diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 8d3c5b1..01ef9d6 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,17 @@ 2016-12-13 Tom Tromey + * python/py-value.c (valpy_dereference, valpy_referenced_value) + (valpy_reference_value, valpy_const_value, valpy_get_address) + (valpy_get_dynamic_type, valpy_lazy_string, valpy_do_cast) + (valpy_getitem, valpy_call, valpy_binop_throw, valpy_negative) + (valpy_absolute, valpy_richcompare_throw): Use scoped_value_mark. + * dwarf2loc.c (dwarf2_loc_desc_get_symbol_read_needs): Use + scoped_value_mark. + * dwarf2-frame.c (execute_stack_op): Use scoped_value_mark. + * value.h (scoped_value_mark): New class. + +2016-12-13 Tom Tromey + * dwarf2read.c (dwarf2_build_psymtabs): Use psymtab_discarder. * psympriv.h (make_cleanup_discard_psymtabs): Don't declare. * psymtab.c (discard_psymtabs_upto): Remove. diff --git a/gdb/dwarf2-frame.c b/gdb/dwarf2-frame.c index beab304..93647ab 100644 --- a/gdb/dwarf2-frame.c +++ b/gdb/dwarf2-frame.c @@ -403,10 +403,9 @@ execute_stack_op (const gdb_byte *exp, ULONGEST len, int addr_size, CORE_ADDR initial, int initial_in_stack_memory) { CORE_ADDR result; - struct cleanup *old_chain; dwarf_expr_executor ctx; - old_chain = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; ctx.this_frame = this_frame; ctx.gdbarch = get_frame_arch (this_frame); @@ -430,8 +429,6 @@ execute_stack_op (const gdb_byte *exp, ULONGEST len, int addr_size, Not implemented: computing unwound register using explicit value operator")); } - do_cleanups (old_chain); - return result; } diff --git a/gdb/dwarf2loc.c b/gdb/dwarf2loc.c index d19f6a2..f5ab371 100644 --- a/gdb/dwarf2loc.c +++ b/gdb/dwarf2loc.c @@ -2801,16 +2801,14 @@ dwarf2_loc_desc_get_symbol_read_needs (const gdb_byte *data, size_t size, struct dwarf2_per_cu_data *per_cu) { int in_reg; - struct cleanup *old_chain; struct objfile *objfile = dwarf2_per_cu_objfile (per_cu); + scoped_value_mark free_values; + symbol_needs_eval_context ctx; ctx.needs = SYMBOL_NEEDS_NONE; ctx.per_cu = per_cu; - - old_chain = make_cleanup_value_free_to_mark (value_mark ()); - ctx.gdbarch = get_objfile_arch (objfile); ctx.addr_size = dwarf2_per_cu_addr_size (per_cu); ctx.ref_addr_size = dwarf2_per_cu_ref_addr_size (per_cu); @@ -2831,8 +2829,6 @@ dwarf2_loc_desc_get_symbol_read_needs (const gdb_byte *data, size_t size, in_reg = 1; } - do_cleanups (old_chain); - if (in_reg) ctx.needs = SYMBOL_NEEDS_FRAME; return ctx.needs; diff --git a/gdb/python/py-value.c b/gdb/python/py-value.c index c1f3e16..f7ec202 100644 --- a/gdb/python/py-value.c +++ b/gdb/python/py-value.c @@ -178,11 +178,10 @@ valpy_dereference (PyObject *self, PyObject *args) TRY { struct value *res_val; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; res_val = value_ind (((value_object *) self)->value); result = value_to_value_object (res_val); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -209,7 +208,7 @@ valpy_referenced_value (PyObject *self, PyObject *args) TRY { struct value *self_val, *res_val; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; self_val = ((value_object *) self)->value; switch (TYPE_CODE (check_typedef (value_type (self_val)))) @@ -226,7 +225,6 @@ valpy_referenced_value (PyObject *self, PyObject *args) } result = value_to_value_object (res_val); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -247,12 +245,10 @@ valpy_reference_value (PyObject *self, PyObject *args) TRY { struct value *self_val; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; self_val = ((value_object *) self)->value; result = value_to_value_object (value_ref (self_val)); - - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -273,13 +269,11 @@ valpy_const_value (PyObject *self, PyObject *args) TRY { struct value *self_val, *res_val; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; self_val = ((value_object *) self)->value; res_val = make_cv_value (1, 0, self_val); result = value_to_value_object (res_val); - - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -301,12 +295,10 @@ valpy_get_address (PyObject *self, void *closure) TRY { struct value *res_val; - struct cleanup *cleanup - = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; res_val = value_addr (val_obj->value); val_obj->address = value_to_value_object (res_val); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -354,7 +346,7 @@ valpy_get_dynamic_type (PyObject *self, void *closure) TRY { struct value *val = obj->value; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; type = value_type (val); type = check_typedef (type); @@ -387,8 +379,6 @@ valpy_get_dynamic_type (PyObject *self, void *closure) /* Re-use object's static type. */ type = NULL; } - - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -428,7 +418,7 @@ valpy_lazy_string (PyObject *self, PyObject *args, PyObject *kw) TRY { - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; if (TYPE_CODE (value_type (value)) == TYPE_CODE_PTR) value = value_ind (value); @@ -436,8 +426,6 @@ valpy_lazy_string (PyObject *self, PyObject *args, PyObject *kw) str_obj = gdbpy_create_lazy_string_object (value_address (value), length, user_encoding, value_type (value)); - - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -514,7 +502,7 @@ valpy_do_cast (PyObject *self, PyObject *args, enum exp_opcode op) { struct value *val = ((value_object *) self)->value; struct value *res_val; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; if (op == UNOP_DYNAMIC_CAST) res_val = value_dynamic_cast (type, val); @@ -527,7 +515,6 @@ valpy_do_cast (PyObject *self, PyObject *args, enum exp_opcode op) } result = value_to_value_object (res_val); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -737,8 +724,8 @@ valpy_getitem (PyObject *self, PyObject *key) TRY { struct value *tmp = self_value->value; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); struct value *res_val = NULL; + scoped_value_mark free_values; if (field) res_val = value_struct_elt (&tmp, NULL, field.get (), NULL, @@ -783,7 +770,6 @@ valpy_getitem (PyObject *self, PyObject *key) if (res_val) result = value_to_value_object (res_val); - do_cleanups (cleanup); } CATCH (ex, RETURN_MASK_ALL) { @@ -861,12 +847,11 @@ valpy_call (PyObject *self, PyObject *args, PyObject *keywords) TRY { - struct cleanup *cleanup = make_cleanup_value_free_to_mark (mark); + scoped_value_mark free_values; struct value *return_value; return_value = call_function_by_hand (function, args_count, vargs); result = value_to_value_object (return_value); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -1014,11 +999,12 @@ valpy_binop_throw (enum valpy_opcode opcode, PyObject *self, PyObject *other) PyObject *result = NULL; struct value *arg1, *arg2; - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); struct value *res_val = NULL; enum exp_opcode op = OP_NULL; int handled = 0; + scoped_value_mark free_values; + /* If the gdb.Value object is the second operand, then it will be passed to us as the OTHER argument, and SELF will be an entirely different kind of object, altogether. Because of this, we can't @@ -1026,17 +1012,11 @@ valpy_binop_throw (enum valpy_opcode opcode, PyObject *self, PyObject *other) python as well. */ arg1 = convert_value_from_python (self); if (arg1 == NULL) - { - do_cleanups (cleanup); - return NULL; - } + return NULL; arg2 = convert_value_from_python (other); if (arg2 == NULL) - { - do_cleanups (cleanup); - return NULL; - } + return NULL; switch (opcode) { @@ -1130,7 +1110,6 @@ valpy_binop_throw (enum valpy_opcode opcode, PyObject *self, PyObject *other) if (res_val) result = value_to_value_object (res_val); - do_cleanups (cleanup); return result; } @@ -1209,12 +1188,11 @@ valpy_negative (PyObject *self) TRY { /* Perhaps overkill, but consistency has some virtue. */ - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; struct value *val; val = value_neg (((value_object *) self)->value); result = value_to_value_object (val); - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -1239,12 +1217,10 @@ valpy_absolute (PyObject *self) TRY { - struct cleanup *cleanup = make_cleanup_value_free_to_mark (value_mark ()); + scoped_value_mark free_values; if (value_less (value, value_zero (value_type (value), not_lval))) isabs = 0; - - do_cleanups (cleanup); } CATCH (except, RETURN_MASK_ALL) { @@ -1362,15 +1338,14 @@ valpy_richcompare_throw (PyObject *self, PyObject *other, int op) int result; struct value *value_other; struct value *value_self; - struct value *mark = value_mark (); struct cleanup *cleanup; + scoped_value_mark free_values; + value_other = convert_value_from_python (other); if (value_other == NULL) return -1; - cleanup = make_cleanup_value_free_to_mark (mark); - value_self = ((value_object *) self)->value; switch (op) @@ -1403,7 +1378,6 @@ valpy_richcompare_throw (PyObject *self, PyObject *other, int op) break; } - do_cleanups (cleanup); return result; } diff --git a/gdb/value.h b/gdb/value.h index f776323..3a0641a 100644 --- a/gdb/value.h +++ b/gdb/value.h @@ -714,6 +714,28 @@ extern struct value *value_mark (void); extern void value_free_to_mark (const struct value *mark); +/* A helper class that uses value_mark at construction time and calls + value_free_to_mark in the destructor. This is used to clear out + temporary values created during the lifetime of this object. */ +class scoped_value_mark +{ + public: + + scoped_value_mark () + : m_value (value_mark ()) + { + } + + ~scoped_value_mark () + { + value_free_to_mark (m_value); + } + + private: + + const struct value *m_value; +}; + extern struct value *value_cstring (const char *ptr, ssize_t len, struct type *char_type); extern struct value *value_string (const char *ptr, ssize_t len,