[PATCH] D115038: [runtimes][VE] Change to compile crtbegin/end before compiler-rt

Kazushi Marukawa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 3 19:20:44 PST 2021


kaz7 abandoned this revision.
kaz7 added a comment.

In D115038#3170172 <https://reviews.llvm.org/D115038#3170172>, @phosek wrote:

> In D115038#3169582 <https://reviews.llvm.org/D115038#3169582>, @kaz7 wrote:
>
>> This patch add mechanism to compile crtbegin/end before other libraries which is similar to a mechanism to compile builtins before compile-rt.
>
> Thank you for this change. We've discussed this before with other contributors. The main concern over adding another sub-build is the increased overhead since every CMake run takes considerable time. To address that, I have implemented D89492 <https://reviews.llvm.org/D89492> which makes it possible to build builtins and crtbegin/crtend in a single CMake build without requiring another sub-build. That change is ready to land but it's been waiting for someone to review it, do you want to take a look if it addresses your use case?

Thank you for trying to achieve pre-compiling of crtbegin/crtend and builtins.  I'll join D89492 <https://reviews.llvm.org/D89492> and review it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115038/new/

https://reviews.llvm.org/D115038



More information about the llvm-commits mailing list