This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH 03/08] PR jit/63854: Fix leak in real.c for i386:init_ext_80387_constants


Valgrind of testsuite/jit.dg/test-types.c showed this leak in real.c
when converting the strings in init_ext_80387_constants to real.

160 bytes in 5 blocks are definitely lost in loss record 89 of 144
   at 0x4A0645D: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x30AF40C368: __gmp_default_allocate (memory.c:44)
   by 0x309842EA20: mpfr_init2 (init2.c:55)
   by 0x527C033: real_from_string(real_value*, char const*) (real.c:2045)
   by 0x56F0CEF: init_ext_80387_constants() (i386.c:9200)
   by 0x56F0E4A: standard_80387_constant_p(rtx_def*) (i386.c:9237)
   by 0x5740115: ix86_rtx_costs(rtx_def*, int, int, int, int*, bool) (i386.c:41735)
   by 0x52E4417: rtx_cost(rtx_def*, rtx_code, int, bool) (rtlanal.c:3855)
   by 0x4F28E9C: set_src_cost(rtx_def*, bool) (rtl.h:2111)
   by 0x4F33AFB: compress_float_constant(rtx_def*, rtx_def*) (expr.c:3667)
   by 0x4F3380E: emit_move_insn(rtx_def*, rtx_def*) (expr.c:3590)
   by 0x4F3A1B6: store_expr_with_bounds(tree_node*, rtx_def*, int, bool, tree_node*) (expr.c:5540)

Fix it.

gcc/ChangeLog:
	PR jit/63854
	* real.c (real_from_string): Add missing mpfr_clear.
---
 gcc/real.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gcc/real.c b/gcc/real.c
index cebbe8d..5e8050d 100644
--- a/gcc/real.c
+++ b/gcc/real.c
@@ -2067,6 +2067,7 @@ real_from_string (REAL_VALUE_TYPE *r, const char *str)
 	  gcc_assert (r->cl = rvc_normal);
 	  /* Set a sticky bit if mpfr_strtofr was inexact.  */
 	  r->sig[0] |= inexact;
+	  mpfr_clear (m);
 	}
     }
 
-- 
1.8.5.3


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]