[PATCH] Preliminary support for dynamically loadable coff objects
lhames at gmail.com
Sat Feb 21 10:46:10 PST 2015
Thanks very much for working on this. It looks great.
If you plan to support more architectures in the future you should consider moving the target specific code into a subclass (along the same lines as RuntimeDyldMachO). In my experience mixing the relocation handling code for multiple architectures was a debugging and maintenance headache - I ended up expending a lot of effort to untangle that code last year.
Could you also add some tests under llvm/test/ExecutionEngine/RuntimeDyld/ ?
More information about the llvm-commits