diff options
| author | Christian <c@ethdev.com> | 2015-01-07 22:41:11 +0800 |
|---|---|---|
| committer | Christian <c@ethdev.com> | 2015-01-07 23:26:15 +0800 |
| commit | 646f99d34ae42d1fe17590c850e7f1cf999ea8b8 (patch) | |
| tree | bd90c295f26deb0c4a8fb005b837e3857db5febc /SolidityOptimizer.cpp | |
| parent | ecf3c0bb324b328b562408a5605aa33c6bfca9f4 (diff) | |
| download | dexon-solidity-646f99d34ae42d1fe17590c850e7f1cf999ea8b8.tar.gz dexon-solidity-646f99d34ae42d1fe17590c850e7f1cf999ea8b8.tar.zst dexon-solidity-646f99d34ae42d1fe17590c850e7f1cf999ea8b8.zip | |
Some changes to the optimizer.
Diffstat (limited to 'SolidityOptimizer.cpp')
| -rw-r--r-- | SolidityOptimizer.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/SolidityOptimizer.cpp b/SolidityOptimizer.cpp index ef5c6f9b..7e06c2e3 100644 --- a/SolidityOptimizer.cpp +++ b/SolidityOptimizer.cpp @@ -48,7 +48,7 @@ public: m_optimize = true; bytes optimizedBytecode = compileAndRun(_sourceCode, _value, _contractName); int sizeDiff = nonOptimizedBytecode.size() - optimizedBytecode.size(); - BOOST_CHECK_MESSAGE(sizeDiff == int(_expectedSizeDecrease), "Bytecode did only shrink by " + BOOST_CHECK_MESSAGE(sizeDiff == int(_expectedSizeDecrease), "Bytecode shrank by " + boost::lexical_cast<string>(sizeDiff) + " bytes, expected: " + boost::lexical_cast<string>(_expectedSizeDecrease)); m_optimizedContract = m_contractAddress; @@ -91,10 +91,10 @@ BOOST_AUTO_TEST_CASE(large_integers) contract test { function f() returns (uint a, uint b) { a = 0x234234872642837426347000000; - b = 0x110000000000000000000000002; + b = 0x10000000000000000000000002; } })"; - compileBothVersions(28, sourceCode); + compileBothVersions(33, sourceCode); compareVersions(0); } @@ -102,8 +102,8 @@ BOOST_AUTO_TEST_CASE(invariants) { char const* sourceCode = R"( contract test { - function f(uint a) returns (uint b) { - return (((a + (1 - 1)) ^ 0) | 0) & (uint(0) - 1); + function f(int a) returns (int b) { + return int(0) | (int(1) * (int(0) ^ (0 + a))); } })"; compileBothVersions(28, sourceCode); @@ -127,8 +127,8 @@ BOOST_AUTO_TEST_CASE(unused_expressions) BOOST_AUTO_TEST_CASE(constant_folding_both_sides) { // if constants involving the same associative and commutative operator are applied from both - // sides, the operator should be applied only once, because the expression compiler - // (even in non-optimized mode) pushes literals as late as possible + // sides, the operator should be applied only once, because the expression compiler pushes + // literals as late as possible char const* sourceCode = R"( contract test { function f(uint x) returns (uint y) { |