[PATCH] D59264: [Driver] Support compiler-rt crtbegin.o/crtend.o for Linux
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 12 23:56:50 PDT 2019
MaskRay added inline comments.
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:563
+ crtend = Args.hasArg(options::OPT_shared) || IsPIE || IsStaticPIE ?
+ "crtend_shared" : "crtend";
> MaskRay wrote:
> > I believe `crtbegin.o` `crtend.o` should just work. It is not necessary to use `crtbegin_shared.o` `crtend_shared.o`.
> This is related to your comments on D28791, specifically that we should be using `crtbegin_shared.o` for `-shared` or `-pie` and `crtbegin.o` otherwise, is that not the case?
Yes. I think we can rename `crtbegin_shared.o` to `crtbegin.o` and use it for every configuration: `-no-pie` `-pie` `-shared` `-static` `-static -pie`.
CHANGES SINCE LAST ACTION
More information about the cfe-commits