Re: [PATCH] D13913: Allow linking multiple bitcode files.

2015-10-27 Thread Artem Belevich via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251427: Allow linking multiple bitcode files. (authored by tra). Changed prior to commit: http://reviews.llvm.org/D13913?vs=38046&id=38562#toc Repository: rL LLVM http://reviews.llvm.org/D13913 Fil

Re: [PATCH] D13913: Allow linking multiple bitcode files.

2015-10-21 Thread Artem Belevich via cfe-commits
tra updated this revision to Diff 38046. tra added a comment. Added a comment about format of LinkModules vector. Added/cleaned up few comments in test files. http://reviews.llvm.org/D13913 Files: include/clang/Basic/LangOptions.def include/clang/CodeGen/CodeGenAction.h include/clang/Driv

Re: [PATCH] D13913: Allow linking multiple bitcode files.

2015-10-21 Thread Artem Belevich via cfe-commits
tra marked an inline comment as done. tra added a comment. http://reviews.llvm.org/D13913 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13913: Allow linking multiple bitcode files.

2015-10-21 Thread Eric Christopher via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. One inline comment, otherwise LGTM. -eric Comment at: include/clang/CodeGen/CodeGenAction.h:28 @@ -27,3 +27,3 @@ std::unique_ptr TheModule; - llvm::Module *LinkModule

[PATCH] D13913: Allow linking multiple bitcode files.

2015-10-20 Thread Artem Belevich via cfe-commits
tra created this revision. tra added reviewers: echristo, pcc. tra added a subscriber: cfe-commits. This will be used for CUDA where we need to link with vendor-supplied bitcode library and with a bitcode library pretending to be libm for GPUs. Linking options for particular file depend on the