From a1630e1d307126653bd75d2a68d3489125b5dc1c Mon Sep 17 00:00:00 2001 From: Martin Whitaker Date: Sat, 10 Jan 2015 17:24:50 +0000 Subject: [PATCH] Fix for br968. __vpiVThrVec4Stack::vpi_get_value_int_ was always treating the thread variable as unsigned, rather than observing the value of __vpiVThrVec4Stack::signed_flag_. Not sure why this was done - none of the regression tests broke when I changed this. --- vvp/vpi_vthr_vector.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/vvp/vpi_vthr_vector.cc b/vvp/vpi_vthr_vector.cc index bc13f13c4..d5b65f3db 100644 --- a/vvp/vpi_vthr_vector.cc +++ b/vvp/vpi_vthr_vector.cc @@ -1,5 +1,5 @@ /* - * Copyright (c) 2001-2014 Stephen Williams (steve@icarus.com) + * Copyright (c) 2001-2015 Stephen Williams (steve@icarus.com) * Copyright (c) 2001 Stephan Boettcher * * This source code is free software; you can redistribute it @@ -433,8 +433,7 @@ void __vpiVThrVec4Stack::vpi_get_value_hexstr_(p_vpi_value vp, const vvp_vector4 void __vpiVThrVec4Stack::vpi_get_value_int_(p_vpi_value vp, const vvp_vector4_t&val) { int32_t vali = 0; - int signed_flag = 0; - vector4_to_value(val, vali, signed_flag, false); + vector4_to_value(val, vali, signed_flag_, false); vp->value.integer = vali; }