Last Comment Bug 746285 - pack nsXBLBinding more carefully for 64-bit
: pack nsXBLBinding more carefully for 64-bit
Product: Core
Classification: Components
Component: XBL (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla15
Assigned To: Nathan Froyd [:froydnj]
Depends on:
  Show dependency treegraph
Reported: 2012-04-17 12:39 PDT by Nathan Froyd [:froydnj]
Modified: 2012-04-24 18:05 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (2.06 KB, patch)
2012-04-17 12:39 PDT, Nathan Froyd [:froydnj]
bzbarsky: review+
Details | Diff | Review

Description Nathan Froyd [:froydnj] 2012-04-17 12:39:48 PDT
Created attachment 615828 [details] [diff] [review]

nsXBLBinding is laid out like so:

  int mRefCnt;
  nsIContent *mBoundElement;
  nsClassHashtable* mInsertionPointTable;
  bool mIsStyleBinding;
  bool mMarkedForDeath;

which introduces padding after mRefCnt and at the end of the structure.  Moving the booleans up into the padding required after mRefCnt saves a word at the end of the structure.
Comment 1 Boris Zbarsky [:bz] 2012-04-17 12:43:26 PDT
Comment on attachment 615828 [details] [diff] [review]

Comment 2 Nathan Froyd [:froydnj] 2012-04-20 13:25:20 PDT
Comment 3 :Ehsan Akhgari (busy, don't ask for review please) 2012-04-24 18:05:20 PDT

Note You need to log in before you can comment on or make changes to this bug.