Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PeekPokeAPI: include source location on failed expect() calls. (backport #4144) #4149

Merged
merged 3 commits into from
Jun 6, 2024
Merged

PeekPokeAPI: include source location on failed expect() calls. (backport #4144) #4149

merged 3 commits into from
Jun 6, 2024

Commits on Jun 6, 2024

  1. PeekPokeAPI: include source location on failed expect() calls. (#4144) 

    * simulator: add SourceInfo to expect calls and report.
* simulator: add test for failed expects.
* simulator: attempt to extract source line.
* simulator: make testableData.expect's sourceInfo parameter explicit.
* simulator: add factory method for giving failed expect sourceInfo/extraContext.

(cherry picked from commit 45dd82a)

# Conflicts:
#	src/test/scala/chiselTests/simulator/SimulatorSpec.scala
    @kivikakk @mergify
    kivikakk authored and mergify[bot] committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    292af69 View commit details
    Browse the repository at this point in the history

  2. Resolve backport conflicts and binary compatibility issues

    @jackkoenig
    jackkoenig committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    26294c9 View commit details
    Browse the repository at this point in the history

  3. Run scalafmt

    @jackkoenig
    jackkoenig committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    4745dc2 View commit details
    Browse the repository at this point in the history