macOS: avoid calling pthread_self() twice#68767
Conversation
|
r? @shepmaster (rust_highfive has picked a reviewer for you, use r? to override) |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
Ping from triage: @kubo39 - the build has failed. |
|
This seems reasonable, but can you share a little about why this change is important or why you noticed it? |
|
You can go ahead and squash your two commits together now as well. |
|
I noticed this while reading the code. To be honest, this is not so important change(maybe slightly faster). |
|
@bors r+ rollup |
|
📌 Commit 67068f3 has been approved by |
|
Thanks! |
macOS: avoid calling pthread_self() twice
macOS: avoid calling pthread_self() twice
macOS: avoid calling pthread_self() twice
macOS: avoid calling pthread_self() twice
macOS: avoid calling pthread_self() twice
Rollup of 8 pull requests Successful merges: - #67272 (recursion_limit parsing handles overflows) - #68597 (Simplify `Skip::nth` and `Skip::last` implementations) - #68767 (macOS: avoid calling pthread_self() twice) - #69175 (Do not ICE when encountering `yield` inside `async` block) - #69223 (Ignore GDB versions with broken str printing.) - #69244 (configure: set LLVM flags with a value) - #69249 (Stabilize {f32, f64}::{LOG2_10, LOG10_2}) - #69252 (Clean out unused directories for extra disk space) Failed merges: r? @ghost
macOS: avoid calling pthread_self() twice
Rollup of 8 pull requests Successful merges: - #67272 (recursion_limit parsing handles overflows) - #68597 (Simplify `Skip::nth` and `Skip::last` implementations) - #68767 (macOS: avoid calling pthread_self() twice) - #69175 (Do not ICE when encountering `yield` inside `async` block) - #69223 (Ignore GDB versions with broken str printing.) - #69244 (configure: set LLVM flags with a value) - #69249 (Stabilize {f32, f64}::{LOG2_10, LOG10_2}) - #69252 (Clean out unused directories for extra disk space) Failed merges: r? @ghost
No description provided.