Hmm, this is rather what I was afraid of. It looks like this has caused nearly all compiler performance tests to regress. Can you paste the full testsuite output?
Tue, Feb 5
Thu, Jan 31
Thanks, looks good!
Yes, I would like to integrate this. I need some pointers on what changes are requested to finish.
Jan 22 2019
Jan 21 2019
Jan 20 2019
Moved to GitLab !156.
Moved to GitLab for CI.
What is the plan for this, @watashi?
@DavidEichmann, what is the status of this?
What is the status of this, @simonmar?
@alansp, I just found this sitting in the merge queue. Unfortunately it doesn't appear to apply and I'm having some trouble resolving the conflicts. Do you suppose you could have a look? My apologies for dropping the ball on this one.
Merged with 18b74cdbbfa053704caabd104587d8a91a9233a0.
@goldfire, I believe this might be blocked on you.
What is the status of this, @carter?
Do you intend on continuing work on this, @mpickering?
This is looking great, @sjakobi! What is your planned timeframe for finishing it up?
What is the status of this, @v0d1ch?
@dfeuer, do you intend on finishing this?
I believe all of the pieces of Trac #14880 have been merged.
The goal of this was to make it easy to track SRT scavenging events to test Trac #15113. However, indeed I'm not convinced this change is worth the extra instructions it would no doubt result in. I've pushed the patch to wip/rts-srt-closure-type for posterity.
I believe all of the pieces of Trac #14880, which this was one, have been merged. See the ticket for details.
It looks like 2257a86daa72db382eb927df12a718669d5491f8 is relevant.
What is the status of this, @goldfire?
@alanz, what is the status of this?
I have rebased and moved this to GitLab !153 for review, CI, and merge.
How is this going, @dminuoso?
@ckoparkar, how is this going?
Bumping out of the review queue while work continues.
I have moved this to GitLab.
Whoops, I found this comment unsubmitted. @andrewthad, what is the status of this?
Merged in a0dcf76fa7b23cf1cacababadaa1b54d90453f9a.
@osa1, what is the status of this?
Merged with c121e33f9b039acf2ac6939af8bfafe593560039.
Merged in 6486c6e49c53e75f37ed732b38c5be7ae64785e8.
@andrewthad, this looks like the beginnings of a nice patch. However, given that we haven't really started review yet do you suppose you could move this to GitLab when you would like review?
Wow, I wasn't aware of this awful little bit of code. Well, given that it exists it does seem reasonable to extend it for musl. However, I can't help but wonder whether there isn't a better way to accomplish this.
@angerman, what is the status of this?
This was merged with 509d5be69c7507ba5d0a5f39ffd1613a59e73eea