On Wed, 2007-08-22 at 13:07 -0600, David Mosberger-Tang wrote:
> Jose,
>
> I checked in a patch to support the tdep_get_func_addr() hook you
> need. I didn't check in the changes to the distance-checking so far.
> Something seems fishy there. In your patch, you check "cur_dist < 0",
> but cur_dist is of type "Elf_W(Addr)", which should be an unsigned
> type, AFAIK, so the check makes no sense.
Sorry me David, what a wanted to was this:
...
cur_dist = (Elf_W (Addr)) (ip - val);
if (unlikely (val < ip))
continue;
...
just a sanity/optimization check,
but you are right I can't use the cur_dist value!
Do you want to send a correct version of the patch?
Thanks for reviewing it!
--
Jose Flavio Aguilar Paulino
Toolchain Team
Software Engineer
IBM - Ltc Brazil
_______________________________________________
Libunwind-devel mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/libunwind-devel
