-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat(es/minifier): Distinguish calls in alias analyzer #6080
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
swc-bump:
- swc_ecma_minifier
max_num: usize, | ||
) -> Result<FxHashSet<Id>, ()> { | ||
) -> Result<FxHashSet<Access>, ()> { | ||
let init = | ||
HashSet::with_capacity_and_hasher(max_num, BuildHasherDefault::<FxHasher>::default()); | ||
ids.into_iter().try_fold(init, |mut res, id| { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GH won't let me comment on the below lines, but is the ranges
var necessary? Seems like this could be a while loop:
let mut i = 0;
while i < ids.len() {
…
i += 1
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Brooooooklyn Can you help with this?
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated review comment generated by auto-rebase script
Description:
This PR improves alias analyzer by distinguishing call and reference, thus reducing the number of identifiers.
Related issue: