Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Advent 2019: Tour of Go Compilers #350

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mohammed90
Copy link
Contributor

I have collected benchmark data for gc, gccgo (-O2 and -O3), and gollvm but opted not to include them because tables aren't neat and images of charts don't show enough details of thee variations.

@AliAlhajji
Copy link

Thanks Mohammed for the time and effort you've put into this article. It's very informative, I didn't know there were as many Go compilers!

I suggest a final run of proofreading to fix some syntax errors. I've catched some of them here:

On a high level, compilers are viewed as a single a single, solid, single-step-worker entity.

we must know on what basis do they compiler certain source code

Copy link
Contributor

@mewmew mewmew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Mohammed,

Thanks for this write-up on the various implementations of Go compilers. It was a joy to read (and review)!

I only found a few typos, other than that, a spectacular write-up :)

Wish you a happy winter holidays.

Cheerful regards,
Robin

content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
content/advent-2019/go-compilers.md Outdated Show resolved Hide resolved
@mohammed90
Copy link
Contributor Author

Thank you @AliAlhajji and @mewmew for your review and kind words!

@mohammed90
Copy link
Contributor Author

@bketelsen, this is supposed to be published today. Anything missing?

@mewmew
Copy link
Contributor

mewmew commented Dec 29, 2019

@bketelsen, this is supposed to be published today. Anything missing?

I'd also love to see this post published, as it's very insightful and gives a good introduction to the Go compiler landscape.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants