Skip to content
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

item-bodies/item-types timings regressed for issue-32062-equality-relations #33889

Closed
nagisa opened this issue May 26, 2016 · 11 comments
Closed
Assignees
Labels
A-type-system Area: Type system I-compiletime Issue: Problems and improvements with respect to compile times. P-high High priority

Comments

@nagisa
Copy link
Member

nagisa commented May 26, 2016

See this and this.

@nagisa nagisa added A-type-system Area: Type system I-slow Issue: Problems and improvements with respect to performance of generated code. labels May 26, 2016
@nagisa
Copy link
Member Author

nagisa commented May 26, 2016

cc @eddyb @nikomatsakis

@jonas-schievink
Copy link
Contributor

cc @Marwes

@Marwes
Copy link
Contributor

Marwes commented May 27, 2016

While testing #33816 on my projects I figured I'd run through this example on that version of rustc as well and it seems it doesn't seem to give this slowdown and given that #33816 seems to be rebased on top of master after this slowdown occurred it might have been fixed by that PR (if only by chance?). I may take a look at this sometime this weekend if I get the time though as it still seems unfortunate. (It would be really nice if perf.rust-lang.org could display the commit hash of each run it does. That would make it much easier to pin down which version introduced a regression).

@nikomatsakis
Copy link
Contributor

Probably the same problem is causing a regr in jld-day15-parser

@nikomatsakis
Copy link
Contributor

triage: P-high

@rust-highfive rust-highfive added the P-high High priority label Jun 2, 2016
@nikomatsakis
Copy link
Contributor

It'd be great to do some sort of bisection to try to narrow down what is causing this.

@pnkfelix pnkfelix self-assigned this Jun 2, 2016
@sanxiyn sanxiyn added I-compiletime Issue: Problems and improvements with respect to compile times. and removed I-slow Issue: Problems and improvements with respect to performance of generated code. labels Jun 3, 2016
@eddyb
Copy link
Member

eddyb commented Jun 4, 2016

@nikomatsakis Doesn't look like it, the regression is in translation for jld-day15-parser.

@arielb1
Copy link
Contributor

arielb1 commented Jun 4, 2016

The problematic PR here is 75e23e1 (#33137). Seems logical - that probably broke the caching

@eddyb
Copy link
Member

eddyb commented Jun 4, 2016

@arielb1 So the solution is to move forward with #33816?

@arielb1
Copy link
Contributor

arielb1 commented Jun 4, 2016

That sounds like it.

@arielb1
Copy link
Contributor

arielb1 commented Jun 5, 2016

This is fixed as #33816 landed.

@arielb1 arielb1 closed this as completed Jun 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-type-system Area: Type system I-compiletime Issue: Problems and improvements with respect to compile times. P-high High priority
Projects
None yet
Development

No branches or pull requests

9 participants