Changeset 121374 in webkit for trunk/Source/JavaScriptCore


Ignore:
Timestamp:
Jun 27, 2012, 2:45:08 PM (13 years ago)
Author:
[email protected]
Message:

x86 disassembler confuses immediates with addresses
https://bugs.webkit.org/show_bug.cgi?id=90099

Reviewed by Mark Hahnenberg.

Prepend "$" to immediates to disambiguate between immediates and addresses. This is in
accordance with the gas and AT&T syntax.

  • disassembler/udis86/udis86_syn-att.c:

(gen_operand):

Location:
trunk/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r121372 r121374  
     12012-06-27  Filip Pizlo  <[email protected]>
     2
     3        x86 disassembler confuses immediates with addresses
     4        https://bugs.webkit.org/show_bug.cgi?id=90099
     5
     6        Reviewed by Mark Hahnenberg.
     7       
     8        Prepend "$" to immediates to disambiguate between immediates and addresses. This is in
     9        accordance with the gas and AT&T syntax.
     10
     11        * disassembler/udis86/udis86_syn-att.c:
     12        (gen_operand):
     13
    1142012-06-27  Filip Pizlo  <[email protected]>
    215
  • trunk/Source/JavaScriptCore/disassembler/udis86/udis86_syn-att.c

    r120745 r121374  
    110110        if ( sext_size < 64 )
    111111            sext_mask = ( 1ull << sext_size ) - 1;
    112         mkasm( u, "0x" FMT64 "x", imm & sext_mask );
     112        mkasm( u, "$0x" FMT64 "x", imm & sext_mask );
    113113
    114114                break;
Note: See TracChangeset for help on using the changeset viewer.