[PATCH] MCJIT MachO debugging support

Lang Hames lhames at gmail.com
Mon Jun 16 13:30:00 PDT 2014


Hi Keno,

The updatesSectionAddress implementation is a bit funky, but I don't have any better ideas for how to do it.

Regarding the comment:
 // TODO: Refactor RuntimeDyld.cpp, so we at least don't nede the symbol
 // thing anymore.

Any chance you could expand on how you see this being refactored in the future? Could you file a bug for it and add a reference to the TODO?

Otherwise this looks good to me.
- Lang.

http://reviews.llvm.org/D4005






More information about the llvm-commits mailing list