cerc-io/solidity
10
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Disable optimizeStackAllocation when --no-optimize-yul is used for consistency with StandardCompiler

Browse Source
This commit is contained in:
Kamil Śliwak 2023-05-24 14:46:06 +02:00
parent 477b621f2e
commit ce76472d49
2 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Changelog.md
View File

@ -13,6 +13,7 @@ Compiler Features:
Bugfixes: Bugfixes:
* Commandline Interface: ``--no-optimize-yul`` would not disable optimized stack allocation (unlike setting ``settings.optimizer.details.yul`` to ``false`` in Standard JSON).
* Commandline Interface: It is no longer possible to specify both ``--optimize-yul`` and ``--no-optimize-yul`` at the same time. * Commandline Interface: It is no longer possible to specify both ``--optimize-yul`` and ``--no-optimize-yul`` at the same time.
* SMTChecker: Fix encoding of side-effects inside ``if`` and ``ternary conditional``statements in the BMC engine. * SMTChecker: Fix encoding of side-effects inside ``if`` and ``ternary conditional``statements in the BMC engine.

1
solc/CommandLineParser.cpp
View File

@ -265,6 +265,7 @@ OptimiserSettings CommandLineOptions::optimiserSettings() const
if (optimizer.noOptimizeYul) if (optimizer.noOptimizeYul)
settings.runYulOptimiser = false; settings.runYulOptimiser = false;
settings.optimizeStackAllocation = settings.runYulOptimiser;
if (optimizer.expectedExecutionsPerDeployment.has_value()) if (optimizer.expectedExecutionsPerDeployment.has_value())
settings.expectedExecutionsPerDeployment = optimizer.expectedExecutionsPerDeployment.value(); settings.expectedExecutionsPerDeployment = optimizer.expectedExecutionsPerDeployment.value();