[PATCH] D28791: [compiler-rt][crt] Simple crtbegin and crtend implementation

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 12:57:23 PST 2017


phosek added a comment.

In https://reviews.llvm.org/D28791#663780, @joerg wrote:

> I still say the patch in this form is doing a lot more harm than good. Without a functional version of crtbegin.c/crtend.o, I absolutely object to this change.


Can you elaborate on what do you mean by functional? In the current form, it's minimal but it's sufficient for systems like Linux. I'm happy to extend it to handle other aspects like .ctors/.dtors, but I thought it'd be better to handle it incrementally.


Repository:
  rL LLVM

https://reviews.llvm.org/D28791





More information about the llvm-commits mailing list