[llvm-commits] [PATCH] Add basic ELF Dyld loader (on behalf of Andy Kaylor)

[Rafael Ávila de Espíndola]

On 08/12/11 01:09 PM, Malea, Daniel wrote:
> Hi all,
> 
>  
> 
> There was a bug (related to symbol address computation) in the patch I
> posted here earlier. If anyone’s interested in a very minimal runtime
> ELF loader (with a workaround for the aforementioned bug) please take a
> look at the attached patch.
> 

Nit:

+      case sys::COFF_FileType:
+      default:

If you are going to use a default you don't need to list
sys::COFF_FileType. Alternatively, drop the default so that we get an
warning when someone adds something to the enum.

Is this sufficient to load a small program? Say "int main() {return
42;}"? Can you include a test?

Thanks a lot for working on this!

> 
> Thanks,
> 
> Daniel
> 
>  

Cheers,
Rafael