Skip to content

Conversation

jonmeow
Copy link
Contributor

@jonmeow jonmeow commented May 29, 2025

Where --no-dump-sem-ir is used, change to --dump-sem-ir-ranges=only. Otherwise, add --dump-sem-ir-ranges=if-present with a TODO to change to only.

Note, SemIR is affected just because the extra comments change line numbers in files where splits aren't in use.

@github-actions github-actions bot requested a review from geoffromer May 29, 2025 18:26
@dwblaikie dwblaikie added this pull request to the merge queue May 29, 2025
Merged via the queue into carbon-language:trunk with commit 2e3fb4d May 29, 2025
10 checks passed
@jonmeow jonmeow deleted the if-present-namespace branch May 29, 2025 19:19
hamphet pushed a commit to hamphet/carbon-lang that referenced this pull request May 29, 2025
Where `--no-dump-sem-ir` is used, change to `--dump-sem-ir-ranges=only`.
Otherwise, add `--dump-sem-ir-ranges=if-present` with a TODO to change
to `only`.

Note, SemIR is affected just because the extra comments change line
numbers in files where splits aren't in use.
chandlerc pushed a commit to chandlerc/carbon-lang that referenced this pull request May 31, 2025
Where `--no-dump-sem-ir` is used, change to `--dump-sem-ir-ranges=only`.
Otherwise, add `--dump-sem-ir-ranges=if-present` with a TODO to change
to `only`.

Note, SemIR is affected just because the extra comments change line
numbers in files where splits aren't in use.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants