Bug 256085 - [Memory] Re-order FontSizeAdjust members to remove some padding
Summary: [Memory] Re-order FontSizeAdjust members to remove some padding
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-04-28 03:18 PDT by Matthieu Dubet
Modified: 2023-04-28 10:17 PDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matthieu Dubet 2023-04-28 03:18:36 PDT 
Current layout is:

  +0 < 12> WebCore::FontSizeAdjust
  +0 <  1>   WebCore::FontSizeAdjust::Metric metric
  +1 <  3>   <PADDING: 3 bytes>
  +4 <  4>     WTF::Markable<float, WebCore::FloatMarkableTraits> value
  +4 <  4>       float m_value
  +8 <  1>   bool isFromFont
  +9 <  3>   <PADDING: 3 bytes>
Total byte size: 12
Total pad bytes: 6
Padding percentage: 50.00 %
Comment 1 Radar WebKit Bug Importer 2023-04-28 03:19:10 PDT 
<rdar://problem/108652215>
Comment 2 Matthieu Dubet 2023-04-28 03:27:30 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/13268
Comment 3 EWS 2023-04-28 10:16:58 PDT 
Committed 263504@main (30d90238fd84): <https://commits.webkit.org/263504@main>

Reviewed commits have been landed. Closing PR #13268 and removing active labels.