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.

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

Exports TXmlNode and the k TXmlNodeKind field. #511

Closed
wants to merge 1 commit into from
Closed

Exports TXmlNode and the k TXmlNodeKind field. #511

wants to merge 1 commit into from

Conversation

gradha
Copy link
Contributor

@gradha gradha commented Jul 5, 2013

Iterators like items will assert on bad input, but end user code
can't check the type beforehand because the kind field is not exported.

Iterators like ``items`` will assert on bad input, but end user code
can't check the type beforehand because the kind field is not exported.
@gradha gradha closed this Jul 5, 2013
@gradha gradha deleted the pr_opens_txmlnode branch July 5, 2013 20:32
Clyybber pushed a commit to Clyybber/Nim that referenced this pull request Sep 16, 2023
511: add a simple tracing framework r=zerbina a=zerbina

## Summary
The goal is to provide the base for an event-based tracing framework
that can be incrementally improved/expanded upon. In its current form,
it's mainly meant for execution-time tracing.

While generally tailored towards being used as part of the compiler,
it's meant to be general enough to be used for both one-off temporary
instrumentation for use during debugging, and for building a persistent,
maybe even user-facing, compiler tracing facility. For example, it could
be expanded to also support the tracing done by `debugutils`.

This commit only adds the framework and a JSON serializer - both are not
used anywhere in the compiler yet.

## Details
The `tracer` module implements the data types and routines that make up
the framework.

The `trace_dump` module implements a serializer to the "Chrome Trace
Event" JSON Object format, which was chosen due to it being
simple-yet-powerful, with multiple visualizers already existing for it.

Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant