Skip to content

stacktraces: Add an extension point for printing custom-owner CIs#58430

Merged
Keno merged 1 commit intomasterfrom
kf/customcistacktrace
May 16, 2025
Merged

stacktraces: Add an extension point for printing custom-owner CIs#58430
Keno merged 1 commit intomasterfrom
kf/customcistacktrace

Conversation

@Keno
Copy link
Member

@Keno Keno commented May 16, 2025

On Julia master, it is possible to add custom CodeInstances with non nothing owner to the runtime system (e.g. by invoking them or with custom AbstractInterpreters). However, the backtrace printing for these custom code instances was not in any way distinguished from the backtrace printing for an ordinary code instance of the same method instance, making it sometimes hard to determine what code instance a particular backtrace was referring to (if the downstream compiler was making many different code instances for a particular MethodInstance). Remidy this by adding an explicit extension point that can be overwritten to modify the printing of these custom-owner CodeInstances.

…tances

On Julia master, it is possible to add custom CodeInstances with non `nothing`
owner to the runtime system (e.g. by invoking them or with custom AbstractInterpreters).
However, the backtrace printing for these custom code instances was not in any
way distinguished from the backtrace printing for an ordinary code instance of
the same method instance, making it sometimes hard to determine what code instance
a particular backtrace was referring to (if the downstream compiler was making many
different code instances for a particular MethodInstance). Remidy this by adding
an explicit extension point that can be overwritten to modify the printing of these
custom-owner CodeInstances.
Copy link
Member

@vchuravy vchuravy left a comment

Choose a reason for hiding this comment

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

The only question I have is if we should print something different by default?

@Keno
Copy link
Member Author

Keno commented May 16, 2025

I don't think I want to force people to implement this interface if they don't want to, so I think keeping the default unmodified is fine for now. We can revisit.

@Keno Keno merged commit ee7e814 into master May 16, 2025
4 of 7 checks passed
@Keno Keno deleted the kf/customcistacktrace branch May 16, 2025 19:52
DilumAluthge pushed a commit that referenced this pull request Jan 19, 2026
…60740)

This backports the part of #60645
that is applicable to 1.12. Furthermore, I added a type assertion two
lines above the other changes that was taken from
#58430. Without this, there are
still JET warnings in 1.12 for the above query.

cc @DilumAluthge
DilumAluthge pushed a commit that referenced this pull request Jan 19, 2026
…60740)

This backports the part of #60645
that is applicable to 1.12. Furthermore, I added a type assertion two
lines above the other changes that was taken from
#58430. Without this, there are
still JET warnings in 1.12 for the above query.

cc @DilumAluthge
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.

2 participants