[PATCH] D21857: [Driver][OpenMP] Add support to create jobs for unbundling actions.

[Jonas Hahnfeld via cfe-commits]

Hahnfeld added a comment.

Hi Samuel,

this looks pretty good overall!

I've only encountered one issue which I don't really know is caused by which patch: I can't use the (exact) same triple for offloading and the host when compiling separately. I found out that the device object file is taken as input for the final link step and `ld` therefore complains

  <...>/lib64/crt1.o: In function `_start':
  (.text+0x20): undefined reference to `main'

I've not yet found the reason - are there some `unordered_maps` that don't keep the order? I'm also seeing that the first argument to `clang-offload-bundler` is not always for the host...

Thanks,
Jonas


http://reviews.llvm.org/D21857