Optional<UniqueRef<..>> is not only semantically weird (UniqueRef is basically a unique_ptr that promises not to be null), but also inefficient, wasting 8 bytes for the boolean in Optional. It is a pattern that appears throughout the AST. In this patch I start by removing it in a fairly simple spot: the initializer for VariableDeclaration.
Created attachment 373279 [details] Patch
Thanks for the review!
Comment on attachment 373279 [details] Patch Clearing flags on attachment: 373279 Committed r247062: <https://trac.webkit.org/changeset/247062>
All reviewed patches have been landed. Closing bug.
<rdar://problem/52534524>