evm.codes icon indicating copy to clipboard operation
evm.codes copied to clipboard

Feature: adding `Huff` language support to playground code editor.

Open Saw-mon-and-Natalie opened this issue 2 years ago • 12 comments

This PR adds support for Huff language (a low-level EVM language).

  • [x] yarn lint
  • [x] yarn dev functional

Saw-mon-and-Natalie avatar Jun 22 '22 00:06 Saw-mon-and-Natalie

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
evm-codes ✅ Ready (Inspect) Visit Preview Jun 22, 2022 at 0:30AM (UTC)

vercel[bot] avatar Jun 22 '22 00:06 vercel[bot]

Thanks for the initiative! I know there are few people who would be happy with this addition, let us look at this in the next couple of days.

D

cc: @malexw @peroket

dorlevi avatar Jun 22 '22 00:06 dorlevi

Thank you for the PR ;-)

I must admit I am discovering Huff with this PR, and I would love to know how you are planning to use this new feature with evm.codes.

I will also let @malexw to evaluate it technically.

@peroket , Huff like other low-level languages (YUL) allows more direct access to EVM opcodes which can be used to optimize for gas consumption. It also includes some higher-level syntax like macros which allows writing huff programs more efficiently.

Adding support for this language to evm.codes would be a great resource for teaching how to use the language and also to onboard new developers.

Saw-mon-and-Natalie avatar Jun 23 '22 11:06 Saw-mon-and-Natalie

@Saw-mon-and-Natalie : any news on getting https://github.com/huff-language/huffc/pull/40 in ? (we would like to merge this change but prefer to do so after this is in and the dependency is changed)

dorlevi avatar Jun 29 '22 22:06 dorlevi

@dorlevi amazing. I'll ping the team Huff.

Saw-mon-and-Natalie avatar Jun 29 '22 22:06 Saw-mon-and-Natalie

Sure thing, hope you understand why we want to wait for that to get resolved, and thanks again for contributing.

dorlevi avatar Jun 29 '22 22:06 dorlevi

@Saw-mon-and-Natalie Any news on merging the huf change?

dorlevi avatar Jul 12 '22 06:07 dorlevi

@Saw-mon-and-Natalie Any news on merging the huf change?

Hi @dorlevi 👋🏽,

My PR was never reviewed on the huffc repo and the repo is now archived. In order to use the new implementation in Rust huff-rs, We/I would need to submit a new PR to the new repo to add a wasm target.

Saw-mon-and-Natalie avatar Jul 12 '22 21:07 Saw-mon-and-Natalie

@Saw-mon-and-Natalie Any news on merging the huf change?

Hi @dorlevi 👋🏽,

My PR was never reviewed on the huffc repo and the repo is now archived. In order to use the new implementation in Rust huff-rs, We/I would need to submit a new PR to the new repo to add a wasm target.

Thank you for your anwser. Are you going to update your PR with the new one? We can't add a dependency if the repo is not going to be maintained unfortunately.

peroket avatar Jul 13 '22 07:07 peroket

I've actually been working on wasm bindings for the huff compiler. Once we've got them working I'd be happy to submit a PR if you don't mind @Saw-mon-and-Natalie?

ncitron avatar Jul 20 '22 22:07 ncitron

I've actually been working on wasm bindings for the huff compiler. Once we've got them working I'd be happy to submit a PR if you don't mind @Saw-mon-and-Natalie?

@ncitron LFG ☺️

Saw-mon-and-Natalie avatar Jul 20 '22 22:07 Saw-mon-and-Natalie

@Saw-mon-and-Natalie and @ncitron, I'm just doing some housekeeping and wanted to check in on this.

dorlevi avatar Sep 20 '22 13:09 dorlevi

Since there was no update on this in a long while, I'm closing the PR, but we would be happy to get that change if you have some time :-)

peroket avatar May 15 '23 07:05 peroket