Skip to content
This repository was archived by the owner on Jun 24, 2026. It is now read-only.

Improve concretize log messages#1686

Merged
ehennenfent merged 2 commits into
masterfrom
concretize-log-message
May 4, 2020
Merged

Improve concretize log messages#1686
ehennenfent merged 2 commits into
masterfrom
concretize-log-message

Conversation

@smoelius

Copy link
Copy Markdown
Contributor

Before this change:

m.c.worker:INFO: [0] Debug Concretize()

After this change:

m.c.worker:INFO: [0] Debug 'Concretize: Concretize PC (Policy: ALL)'

@bradlarsen

Copy link
Copy Markdown

This is great!

@bradlarsen bradlarsen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming CI passes, LGTM!

Comment thread manticore/core/worker.py Outdated
Co-authored-by: Eric Hennenfent <eric.hennenfent@trailofbits.com>
@ehennenfent
ehennenfent merged commit ad5c48b into master May 4, 2020
@ehennenfent
ehennenfent deleted the concretize-log-message branch May 4, 2020 21:30
ekilmer added a commit that referenced this pull request May 18, 2020
* master: (24 commits)
  Fix symbolic syscall arg concretization (#1697)
  Update README, Add Issue Templates (#1685)
  Fix inheritance magic and add plugin enable/disable (#1696)
  Add hashing to constraintSets (#1703)
  Reduce log messages (#1700)
  Updated Proc error message to be more informative (#1704)
  Move states that selfdestruct to the ready queue. (#1699)
  Reduce concrete hash log messages. (#1695)
  Remove Keystone from existing tests (#1684)
  Change URL for WASM spec (#1702)
  Overhaul Linux file emulation (#1673)
  Try getting more coverage from truffle-test (#1694)
  Eliminate deprecation warning in solidity_create_contract. (#1692)
  Fix a few documentation typos in ManticoreBase (#1687)
  Add set_verbosity to the documentation (#1688)
  Adjust z3 options. (#1691)
  Improve concretize log messages (#1686)
  Move truffle to its own CI type (#1693)
  Remove default gas limit (#1689)
  Better argument names in manticoreEVM api (#1652)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants