Skip to content

fix(minifier): avoid minifying +void unknown to NaN and fix typo #8784

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

Conversation

7086cmd
Copy link
Contributor

@7086cmd 7086cmd commented Jan 30, 2025

fixes #8782
This bug appears when we arbitrarily eliminate potential side-effective elements in unary expression.

@7086cmd 7086cmd requested a review from Boshen January 30, 2025 06:56
@7086cmd 7086cmd marked this pull request as ready for review January 30, 2025 06:57
@github-actions github-actions bot added A-minifier Area - Minifier C-bug Category - Bug labels Jan 30, 2025
Copy link
Contributor Author

7086cmd commented Jan 30, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • 0-merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@7086cmd 7086cmd changed the title fix(minifier): avoid minifying +void unknown to NaN resolves #8782. This bug appears when we are arbitrarily eliminating potential side-effective elements in unary expression. fix(minifier): avoid minifying +void unknown to NaN resolves #8782. Jan 30, 2025
@7086cmd 7086cmd changed the title fix(minifier): avoid minifying +void unknown to NaN resolves #8782. fix(minifier): avoid minifying +void unknown to NaN Jan 30, 2025
Copy link

codspeed-hq bot commented Jan 30, 2025

CodSpeed Performance Report

Merging #8784 will not alter performance

Comparing 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression (ae7f670) with main (fb5b1fa)

Summary

✅ 33 untouched benchmarks

@7086cmd 7086cmd force-pushed the 01-30-feat_minifier_implement_variadic_math_methods_in_known_methods branch from ef16e27 to abc84d1 Compare January 30, 2025 08:23
@7086cmd 7086cmd force-pushed the 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression branch from 2a37266 to bb39d43 Compare January 30, 2025 08:24
@7086cmd 7086cmd force-pushed the 01-30-feat_minifier_implement_variadic_math_methods_in_known_methods branch from abc84d1 to 42f7235 Compare January 30, 2025 08:59
@7086cmd 7086cmd force-pushed the 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression branch from bb39d43 to c23a256 Compare January 30, 2025 08:59
@7086cmd 7086cmd linked an issue Jan 30, 2025 that may be closed by this pull request
@7086cmd 7086cmd force-pushed the 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression branch from c23a256 to 735a5b6 Compare January 30, 2025 13:49
@7086cmd 7086cmd changed the base branch from 01-30-feat_minifier_implement_variadic_math_methods_in_known_methods to main January 30, 2025 13:49
@7086cmd 7086cmd changed the title fix(minifier): avoid minifying +void unknown to NaN fix(minifier): avoid minifying +void unknown to NaN and fix typo Jan 30, 2025
@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Jan 30, 2025
Copy link
Member

Boshen commented Jan 30, 2025

Merge activity

  • Jan 30, 9:59 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Jan 30, 10:00 AM EST: A user added this pull request to the Graphite merge queue.
  • Jan 30, 10:06 AM EST: A user merged this pull request with the Graphite merge queue.

…8784)

fixes #8782
This bug appears when we arbitrarily eliminate potential side-effective elements in unary expression.
@Boshen Boshen force-pushed the 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression branch from 3803fd8 to ae7f670 Compare January 30, 2025 15:01
@graphite-app graphite-app bot merged commit ae7f670 into main Jan 30, 2025
25 checks passed
@graphite-app graphite-app bot deleted the 01-30-fix_minifier_avoid_minifying_void_unknown_to_nan_resolves_8782._this_bug_appears_when_we_are_arbitrarily_eliminating_potential_side-effective_elements_in_unary_expression branch January 30, 2025 15:06
@oxc-bot oxc-bot mentioned this pull request Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-merge Merge with Graphite Merge Queue A-minifier Area - Minifier C-bug Category - Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect substitution in minifier for +(void unknown()).
2 participants