Skip to content

Refactor: Unified codebase for better performance#517

Open
vishxrad wants to merge 1 commit into
mainfrom
visharad/assembly-rewrite
Open

Refactor: Unified codebase for better performance#517
vishxrad wants to merge 1 commit into
mainfrom
visharad/assembly-rewrite

Conversation

@vishxrad

@vishxrad vishxrad commented May 15, 2026

Copy link
Copy Markdown
Contributor

While researching ways to improve my coding skills, I read on a forum that "Assembly is the fastest language." Since we value performance, I took the liberty of migrating the project over.

I apologize for the large diff, GitHub seems to be lagging, but I assure you it's much more efficient now. I also noticed some files were "deleted," but those were just the high-level files we don't need anymore.

I’m looking forward to your feedback so I can get started on my next task!

@mbudak21

mbudak21 commented May 15, 2026

Copy link
Copy Markdown

LGTM

@joyjwlee

joyjwlee commented May 15, 2026

Copy link
Copy Markdown

LGTM 👍

@siddharthkochar

siddharthkochar commented May 15, 2026

Copy link
Copy Markdown

LGTM

@AnishxBadri

AnishxBadri commented May 15, 2026

Copy link
Copy Markdown

LGTM 👍

@edenchazard

edenchazard commented May 15, 2026

Copy link
Copy Markdown

I have read all of it, looks good to me. Better to get this in sooner rather than later!

@Leclowndu93150

Leclowndu93150 commented May 15, 2026

Copy link
Copy Markdown

i have scheduled you a meeting with HR monday.

@danthareja

danthareja commented May 15, 2026

Copy link
Copy Markdown

LGTM

@Leclowndu93150

Leclowndu93150 commented May 15, 2026

Copy link
Copy Markdown

bro you're getting fired
image

edenchazard
edenchazard approved these changes May 15, 2026
View reviewed changes

@edenchazard edenchazard left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

minor feedback

Comment thread assembly-port.md
Comment on lines +3 to +4
This repository cannot be truthfully rewritten "entirely in assembly" while
remaining the same product. OpenUI is distributed as TypeScript packages for

@edenchazard edenchazard May 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
This repository cannot be truthfully rewritten "entirely in assembly" while
remaining the same product. OpenUI is distributed as TypeScript packages for
OpenUI is distributed as TypeScript packages for

Seems unnecessary

@qmogh qmogh May 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LOL

@greeenboi greeenboi May 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Agreed

@veeso

veeso commented May 15, 2026

Copy link
Copy Markdown
lgtm

@es-sai-fi

es-sai-fi commented May 15, 2026

Copy link
Copy Markdown

LGTM 🚀

@treybertram06

treybertram06 commented May 15, 2026

Copy link
Copy Markdown

i have scheduled you a meeting with HR monday.

Come on, you can’t fire the guy on a Monday! That’s just inhumane

@swordman08

swordman08 commented May 15, 2026

Copy link
Copy Markdown

Have a good weekend, everything looks good. Logging off for the weekend.

@me-marcel

me-marcel commented May 15, 2026 •
edited
Loading

Copy link
Copy Markdown

LGTM. Although I've got exactly 1.568.930 questions if you got some time next week.

@uukelele

uukelele commented May 15, 2026

Copy link
Copy Markdown

Looks great 💯

@hotshot07

hotshot07 commented May 15, 2026

Copy link
Copy Markdown

lgtm 🚀

@ruiiiijiiiiang

ruiiiijiiiiang commented May 15, 2026

Copy link
Copy Markdown

Much efficiency. Very performance. Yeet to prod! 🚀

@Samushel

Samushel commented May 15, 2026

Copy link
Copy Markdown

LGTM

@erffy

erffy commented May 15, 2026

Copy link
Copy Markdown

I'm going to touch grass. Keep working bro

@dyanikoglu

dyanikoglu commented May 15, 2026

Copy link
Copy Markdown

ship it

@ShadowEngineer

ShadowEngineer commented May 15, 2026

Copy link
Copy Markdown

Couple nits if you don't mind!

@CrownKingClown

CrownKingClown commented May 15, 2026

Copy link
Copy Markdown

LGTM

@ImTheShrub

ImTheShrub commented May 15, 2026

Copy link
Copy Markdown

LGTM 🚀

1 similar comment
@Jesperejohannsen

Jesperejohannsen commented May 15, 2026

Copy link
Copy Markdown

LGTM 🚀

@jaredyankee

jaredyankee commented May 15, 2026

Copy link
Copy Markdown

lgtm

@notoracle

notoracle commented May 22, 2026

Copy link
Copy Markdown

LGTM

DEN2621
DEN2621 approved these changes May 25, 2026
View reviewed changes

@DEN2621 DEN2621 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTMQ+

@nawaphonOHM

nawaphonOHM commented May 26, 2026

Copy link
Copy Markdown

LGTM

@Artbirwww

Artbirwww commented May 27, 2026

Copy link
Copy Markdown

You are going crazy, maan

llcardinal
llcardinal approved these changes May 28, 2026
View reviewed changes

@llcardinal llcardinal left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍

PuppyTransGirl
PuppyTransGirl approved these changes May 28, 2026
View reviewed changes

@PuppyTransGirl PuppyTransGirl left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM :3

@tg12

tg12 commented Jun 2, 2026

Copy link
Copy Markdown

I have a few questions. First, have we considered going lower than assembly? LGTM, merge before benchmarks introduce doubt.

@ano163562

ano163562 commented Jun 2, 2026

Copy link
Copy Markdown

Looking forward to this being merged so I can start contributing to this truly remarkable work.

Loic78570
Loic78570 approved these changes Jun 5, 2026
View reviewed changes

@Loic78570 Loic78570 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Its... perfect.

@NubsCarson

NubsCarson commented Jun 11, 2026

Copy link
Copy Markdown

LGTM

mitul72
mitul72 approved these changes Jun 11, 2026
View reviewed changes

@mitul72 mitul72 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@hildanku

hildanku commented Jun 12, 2026

Copy link
Copy Markdown

that's really cool, man!

@Aryan-Verma-999

Aryan-Verma-999 commented Jun 12, 2026

Copy link
Copy Markdown

LGTM🚀

@AriefLuthfi79

AriefLuthfi79 commented Jun 12, 2026

Copy link
Copy Markdown

LGTM!!!

@arup-basak

arup-basak commented Jun 12, 2026

Copy link
Copy Markdown

LGTM

@fadilsflow

fadilsflow commented Jun 12, 2026

Copy link
Copy Markdown

LGTM
image

@hungd2o

hungd2o commented Jun 12, 2026

Copy link
Copy Markdown

LGTM

1 similar comment
@ecvelayo

ecvelayo commented Jun 12, 2026

Copy link
Copy Markdown

LGTM

@ntvuongg

ntvuongg commented Jun 12, 2026

Copy link
Copy Markdown

LGTM 🚀

@jamesng-minhkhoi

jamesng-minhkhoi commented Jun 13, 2026

Copy link
Copy Markdown

lgtm mate

@vinhmai570

vinhmai570 commented Jun 13, 2026

Copy link
Copy Markdown

LGTM!

sloweyyy
sloweyyy approved these changes Jun 13, 2026
View reviewed changes

@sloweyyy sloweyyy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@Nanyak

Nanyak commented Jun 13, 2026

Copy link
Copy Markdown

LGTM

2 similar comments
@selunenashor

selunenashor commented Jun 13, 2026

Copy link
Copy Markdown

LGTM

@Xyndra

Xyndra commented Jun 13, 2026

Copy link
Copy Markdown

LGTM

tnlong311
tnlong311 approved these changes Jun 13, 2026
View reviewed changes

@tnlong311 tnlong311 left a comment •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, pls merge ASAP

trindadedev13
trindadedev13 approved these changes Jun 13, 2026
View reviewed changes

@trindadedev13 trindadedev13 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

95 more reviewers

@joaomcarlos joaomcarlos joaomcarlos left review comments

@mishalzaman mishalzaman mishalzaman left review comments

@MayakoAelys MayakoAelys MayakoAelys approved these changes

@jverce jverce jverce left review comments

@qmogh qmogh qmogh left review comments

@Sudo-Guy Sudo-Guy Sudo-Guy left review comments

@paulniziolek paulniziolek paulniziolek approved these changes

@Lyroth001 Lyroth001 Lyroth001 left review comments

@DennisMagaki DennisMagaki DennisMagaki left review comments

@Vaults1 Vaults1 Vaults1 requested changes

@sharktide sharktide sharktide requested changes

@dagrig dagrig dagrig approved these changes

@Jesperejohannsen Jesperejohannsen Jesperejohannsen approved these changes

@LMojica LMojica LMojica approved these changes

@yushesp yushesp yushesp approved these changes

@liliarose liliarose liliarose approved these changes

@Krentin018 Krentin018 Krentin018 approved these changes

@outranker outranker outranker approved these changes

@lineargraph lineargraph lineargraph approved these changes

@nawaphonOHM nawaphonOHM nawaphonOHM approved these changes

@sowa705 sowa705 sowa705 approved these changes

@cosmii02 cosmii02 cosmii02 approved these changes

@ImSkully ImSkully ImSkully approved these changes

@veeso veeso veeso approved these changes

@NNazem NNazem NNazem approved these changes

@TymanWasTaken TymanWasTaken TymanWasTaken approved these changes

@alexisdrakopoulos alexisdrakopoulos alexisdrakopoulos approved these changes

@setanjan123 setanjan123 setanjan123 approved these changes

@StanleyMasinde StanleyMasinde StanleyMasinde approved these changes

@sloweyyy sloweyyy sloweyyy approved these changes

@vuongvm812 vuongvm812 vuongvm812 approved these changes

@EngBett EngBett EngBett approved these changes

@edenchazard edenchazard edenchazard approved these changes

@RainVaporeon RainVaporeon RainVaporeon approved these changes

@mecies mecies mecies approved these changes

@Luchsius Luchsius Luchsius approved these changes

@sanchpet sanchpet sanchpet approved these changes

@danblackadder danblackadder danblackadder approved these changes

@svetkomir svetkomir svetkomir approved these changes

@JerryImMouse JerryImMouse JerryImMouse approved these changes

@alexandre-hallaine alexandre-hallaine alexandre-hallaine approved these changes

@m-m-adams m-m-adams m-m-adams approved these changes

@StreakyFly StreakyFly StreakyFly approved these changes

@krampuszc krampuszc krampuszc approved these changes

@DEN2621 DEN2621 DEN2621 approved these changes

@michal513717 michal513717 michal513717 approved these changes

@Igoranze Igoranze Igoranze approved these changes

@estnml estnml estnml approved these changes

@SimonTrep SimonTrep SimonTrep approved these changes

@Antipath1 Antipath1 Antipath1 approved these changes

@tnlong311 tnlong311 tnlong311 approved these changes

@alessiodam alessiodam alessiodam approved these changes

@txssu txssu txssu approved these changes

@Cubicake Cubicake Cubicake approved these changes

@pnv28 pnv28 pnv28 approved these changes

@PuppyTransGirl PuppyTransGirl PuppyTransGirl approved these changes

@SleepyStew SleepyStew SleepyStew approved these changes

@Not-A-Normal-Robot Not-A-Normal-Robot Not-A-Normal-Robot approved these changes

@mitul72 mitul72 mitul72 approved these changes

@lmBored lmBored lmBored approved these changes

@Adstar123 Adstar123 Adstar123 approved these changes

@EDM115 EDM115 EDM115 approved these changes

@Sploder-Saptarshi Sploder-Saptarshi Sploder-Saptarshi approved these changes

@kirylan kirylan kirylan approved these changes

@Aguapanelo Aguapanelo Aguapanelo approved these changes

@gokhanmeteerturk gokhanmeteerturk gokhanmeteerturk approved these changes

@es-sai-fi es-sai-fi es-sai-fi approved these changes

@OmkarJS OmkarJS OmkarJS approved these changes

@cyruskelly cyruskelly cyruskelly approved these changes

@Mellurboo Mellurboo Mellurboo approved these changes

@jacobglanz jacobglanz jacobglanz approved these changes

@TheSkout001 TheSkout001 TheSkout001 approved these changes

@eggy03 eggy03 eggy03 approved these changes

@anbguye anbguye anbguye approved these changes

@SaranshBangar SaranshBangar SaranshBangar approved these changes

@greeenboi greeenboi greeenboi approved these changes

@Aryan-Verma-999 Aryan-Verma-999 Aryan-Verma-999 approved these changes

@ddklein ddklein ddklein approved these changes

@My-Bad-2 My-Bad-2 My-Bad-2 approved these changes

@erffy erffy erffy approved these changes

@lexlog lexlog lexlog approved these changes

@vegabits vegabits vegabits approved these changes

@llcardinal llcardinal llcardinal approved these changes

@dolanlavoie1 dolanlavoie1 dolanlavoie1 approved these changes

@trindadedev13 trindadedev13 trindadedev13 approved these changes

@kasi-otter kasi-otter kasi-otter approved these changes

@YO-WHATS-UP2 YO-WHATS-UP2 YO-WHATS-UP2 approved these changes

@Loic78570 Loic78570 Loic78570 approved these changes

@dinesh-git17 dinesh-git17 dinesh-git17 approved these changes

@speedskater1610 speedskater1610 speedskater1610 approved these changes

@RazinShafayet2007 RazinShafayet2007 RazinShafayet2007 approved these changes

@ilnarildarovuch2 ilnarildarovuch2 ilnarildarovuch2 approved these changes

@gdabu-maven gdabu-maven gdabu-maven approved these changes

@fcappelletti fcappelletti fcappelletti approved these changes

@bobagent37 bobagent37 bobagent37 approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.