Closed Bug 499631 Opened 16 years ago Closed 11 years ago

byteswap code (NS_SWAP32) should use built-in function on MSVC++

Categories

(Core :: XPCOM, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 857957

People

(Reporter: m_kato, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch v0 (obsolete) — Splinter Review
byteswap code (NS_SWAP32) should use built-in function on MSVC++. By replacing with bswap, it reduces a few code size and improve performance. (bswap is twice faster than original calculation.)
Attached patch fixSplinter Review
Attachment #384355 - Attachment is obsolete: true
Attachment #560712 - Flags: review?(benjamin)
Attachment #560712 - Flags: review?(benjamin) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: