search

amiq-consulting / fc4sc

A header only C++11 library for functional coverage
Apache License 2.0
36 stars 15 forks source link

Merge fix for covergroupCoverage #8

Closed apaiva closed 5 years ago

apaiva commented 5 years ago

Merge tool fix to traverse incoming parseRoot instead of mergeDBtree and to account for covergroupCoverage hierarchy layer.

amiq-consulting commented 5 years ago

Hi,

Thank you for the fix. When running locally, I encountered another issue in one of the queries searching for parent element of a coverpointBin element (parse_coverpoint_bins function). I fixed the problem and (as a bonus) refactored all the queries so the code is much clearer now. These updates are available under the following commit: https://github.com/amiq-consulting/fc4sc/commit/d2fdad86afb7ed35d7285c24823e4c3cacf820d1

Best Regards