A VHDL ExpAttribute() can share the base so don't just delete them

2015-12-13 19:00:04 -08:00 · 2015-12-13 19:00:04 -08:00 · ad4b523edc
parent 45fc06cf11
commit ad4b523edc
1 changed files with 3 additions and 1 deletions
--- a/vhdlpp/expression.cc
+++ b/vhdlpp/expression.cc
@ -61,7 +61,9 @@ ExpAttribute::ExpAttribute(ExpName*bas, perm_string nam)

 ExpAttribute::~ExpAttribute()
 {
-      delete base_;
+	/* Different attributes can point to the same base so we cannot delete this here.
+	 * Look at the vhdl_range test with valgrind to see this issue. */
+//      delete base_;
 }

 Expression*ExpAttribute::clone() const