Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

EIP #198 and Issue #185: added bn_128_add benchmark #6263

Merged
merged 1 commit into from
Aug 9, 2017

Conversation

timxor
Copy link

@timxor timxor commented Aug 9, 2017

Hi Parity Team,

This pull request adds a benchmark for ethereum builtins bn_128_add for EIP#198 and Issue#185 on the metropolis-update branch.

I'm am new to this project, so I'm not yet acquainted with the formalities around ensuring that all the tests pass etc., however I'm enthusiastic about contributing. So if someone would be interested in helping me enhance the quality of this pull request, I would love to speak with you~

Thank you =)

Tim (@tcsiwula)

@parity-cla-bot
Copy link

It looks like @tcsiwula hasn'signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, plesae reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

@timxor
Copy link
Author

timxor commented Aug 9, 2017

[clabot:check]

@parity-cla-bot
Copy link

It looks like @tcsiwula signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@timxor timxor changed the title EIP #198: added bn_128_add benchmark EIP #198 and Issue #185: added bn_128_add benchmark Aug 9, 2017
@arkpar arkpar added the A8-looksgood 🦄 Pull request is reviewed well. label Aug 9, 2017
@arkpar
Copy link
Collaborator

arkpar commented Aug 9, 2017

Thank you for the contribution

@arkpar arkpar merged commit 5ec44b1 into openethereum:metropolis-update Aug 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants