Author Topic: ndisasm offset size  (Read 2888 times)

Offline xcross

  • New Member
  • Posts: 1
ndisasm offset size
« on: June 23, 2011, 05:34:50 PM »
Not sure if this is the correct  forum but I didn't see a development forum.

We use ndisam to debug output from our jit compiler. In re-targeting our compiler to x64 I discovered that the offset argument is only a 32 bit value. However, we generate our code in memory so the code locations are 64 bit pointers on x64. Taking a peek at the source, it seems straight forward to change ndisasm offset to a 64 bit type. Has this ever been discussed? Are there any issues I'm overlooking?


Offline Frank Kotler

  • NASM Developer
  • Hero Member
  • *****
  • Posts: 2446
  • Country: us
Re: ndisasm offset size
« Reply #1 on: June 28, 2011, 02:07:34 PM »
For those who don't follow the "developer's list", Chris has submitted a patch (Thanks, Chris!). I don't think it has been committed yet, but it'll be fixed soon...


Offline Cyrill Gorcunov

  • NASM Developer
  • Full Member
  • *****
  • Posts: 176
  • Country: 00
Re: ndisasm offset size
« Reply #2 on: July 07, 2011, 06:23:45 AM »
As far as I remember HPA said some tune up needed for the patch, so I either missed it either Chris didn't send back updated version. Chris?