Kamil Śliwak

Results 536 comments of Kamil Śliwak

Thanks. So I just checked that with `--via-ir` and it compiles just fine, even with more arguments. So switching to 0.8.13 and using the IR codegen should solve the problem...

- `--via-ir` is a CLI option. Hardhat and Truffle use Standard JSON, so the setting is `settings.viaIR` as I mentioned above. - Compiles to Yul first and then Yul to...

`optimizer.runs` is the way you tell the compiler what you care more about: the deployment cost or the runtime cost. The value is the number of times you expect it...

I mean, without seeing any code we can't really say anything just based on that error message. It simply says that the compiler ran out of available EVM slots but...

> I have no idea why the legacy optimizer is changed at all...? Some bits in the legacy codegen are implemented by inserting Yul code compiled with the new pipeline....

This is the last thing missing for approval: https://github.com/ethereum/solidity/pull/13150#discussion_r951688759, i.e. I think it won't handle symlinks. But that should be easy to fix. The other stuff are just simple tweaks...

The PR looks fine to me in the current form but the last thing that is missing is the extra bit of docs (https://github.com/ethereum/solidity/pull/13256#discussion_r923560310). Other than that, we could merge...

I added a bit of docs about the new flag. The PR is complete now.