- User Since
- Dec 7 2016, 2:42 PM (149 w, 2 d)
Thu, Oct 17
Wed, Oct 16
All three parts have been committed, so I am abandoning the original patch.
Tue, Oct 15
Sun, Oct 13
Fri, Oct 11
Rebased patch and addressed review comments.
Thu, Oct 10
Wed, Oct 9
I have uploaded the last part to https://reviews.llvm.org/D68746
Fri, Oct 4
Rebased patch and changed clang-offload-wrapper CMakeLists.txt to use add_clang_tool() rather than add_clang_executable() with a custom install rule.
Wed, Oct 2
Addressed some comments and rebased patch.
Fri, Sep 27
The second part was uploaded to https://reviews.llvm.org/D68166.
Thu, Sep 26
I have uploaded the first part to https://reviews.llvm.org/D68070
Wed, Sep 25
This patch is no longer relevant since it was agreed to split https://reviews.llvm.org/D64943 into pieces in a different way.
I have rebased patch and addressed last Alexey’s comments. If there are no more comments, I propose to split this patch into 3 pieces
Sep 18 2019
Sep 12 2019
- Changed offload entry section name to “omp_offloading_entries”
- Wrapper bit-code now uses start_ omp_offloading_entries/stop_ omp_offloading_entries symbols for accessing offload entry table assuming that these symbols are defined by the linker
- Removed omptargetbegin.o/omptargetend.o objects
Sep 11 2019
Sep 10 2019
Sep 9 2019
Sep 5 2019
I think I have addressed all comments posted so far. Do you have more notes/comments/suggestions?
Sep 4 2019
Aug 31 2019
Removed trailing '.' from error messages and added few additional changes for better error handling.
Addressed review comments.
Aug 30 2019
Aug 29 2019
Aug 28 2019
Rebased and synced up changes with https://reviews.llvm.org/D64943
Looks like there will be no more comments. If so, I will update the first part https://reviews.llvm.org/D65130 which adds clang-offload-wrapper tool with the latest changes.
Aug 27 2019
Added tests for each file type.
Sure. The main unbundling loop looks as follows (see loop on line 745)