Skip to content

Give shared helpers the location of their earliest use - #1740

Merged
ctate merged 1 commit into
mainfrom
ctate/give-shared-helpers-6ef312
Oct 22, 2026
Merged

ctate merged 1 commit into
mainfrom
ctate/give-shared-helpers-6ef312

Conversation

@ctate

@ctate ctate commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator
  • Fixes main: --explain-dynamic reported shared helpers such as %dyn.class.property.0 at the entry file's 1:1 and left out %method.value:%%Error.toString.
  • A shared helper now takes the location of its earliest use, by file and position. That location still does not depend on lowering order.
  • The coverage snapshot changes only in the order of two helpers.

@vercel

vercel Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
scriptc Ready Ready Preview, v0 Oct 22, 2026 3:03am UTC

@ctate
ctate merged commit b0802dc into main Oct 22, 2026
61 of 64 checks passed
@ctate
ctate deleted the ctate/give-shared-helpers-6ef312 branch October 22, 2026 03:08

This branch was successfully deployed

1 active deployment
Preview — d5688d7d Deployed Oct 22, 2026 by vercel[bot]
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