Skip to content

fix: always set optimizer details#7690

Merged
DaniPopes merged 1 commit into
foundry-rs:masterfrom
mattsse:matt/always-set-optimizer-details
Apr 16, 2024
Merged

fix: always set optimizer details#7690
DaniPopes merged 1 commit into
foundry-rs:masterfrom
mattsse:matt/always-set-optimizer-details

Conversation

@mattsse

@mattsse mattsse commented Apr 16, 2024

Copy link
Copy Markdown
Member

reverts #4427

closes #7689

PR #4427 ignores optimizer_details if optimizer is disabled. However, optimizer = false doesn't completely disable optimizations, and in order to so additional optimizer details have to be provided.

@mattsse mattsse added the T-bug Type: bug label Apr 16, 2024
@mattsse mattsse requested a review from klkvr April 16, 2024 20:20
@DaniPopes DaniPopes merged commit 9079fb6 into foundry-rs:master Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-bug Type: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimizer details are ignored if optimizer is disabled

2 participants