Skip to content

Conversation

@jacob-hughes
Copy link
Contributor

No description provided.

softdev.bib Outdated
}
@misc{chromium20cssbug,
title = {Octane benchmark suite},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh, copy paste bug. I'll push a fix

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I'm a little unsure what to call this. Can I use the lengthy title of the bug report?

"base::Optional when compiled with clang leads to memory layout that is incompatible with conservative stack scanning"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question. Let's try that as the title and see how we feel in a day or so!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@ltratt
Copy link
Member

ltratt commented Mar 2, 2023

Please squash.

@jacob-hughes jacob-hughes force-pushed the add_chromium_css_bug branch from 087bbfe to ae982d7 Compare March 2, 2023 19:23
@jacob-hughes
Copy link
Contributor Author

Squashed

@ltratt
Copy link
Member

ltratt commented Mar 2, 2023

bors r+

bors bot added a commit that referenced this pull request Mar 2, 2023
88: Add conservative stack scanning bug found in Chromium r=ltratt a=jacob-hughes



Co-authored-by: Jake Hughes <jh@jakehughes.uk>
@bors
Copy link
Contributor

bors bot commented Mar 2, 2023

Build failed:

@jacob-hughes
Copy link
Contributor Author

jacob-hughes commented Mar 2, 2023

Fixed: 8454058

@ltratt
Copy link
Member

ltratt commented Mar 2, 2023

Please squash.

@jacob-hughes jacob-hughes force-pushed the add_chromium_css_bug branch from 8454058 to 921357c Compare March 2, 2023 19:29
@jacob-hughes
Copy link
Contributor Author

Squashed

@ltratt
Copy link
Member

ltratt commented Mar 2, 2023

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 2, 2023

Build succeeded:

@bors bors bot merged commit 0870596 into softdevteam:master Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants