[JSC] A bit performance improvement for Object.assign by cleaning up code
Created attachment 345438 [details]
Comment on attachment 345438 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=345438&action=review
> + 2. canDoFastPath is not necessary. Restructuring the code to clean up things.
I agree this is a bit ugly, but the reason I wrote it that way so was we don't try to use the Structure* after the source could have potentially transitioned.
I would vote for just loading the structure twice instead of keeping a variable around that points to something that may no longer be source's structure. (LLVM should CSE the structure load anyways)
Committed r234058: <https://trac.webkit.org/changeset/234058>