[PATCH] D71393: Default to -fuse-init-array

2019-12-12 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay marked an inline comment as done. MaskRay added inline comments. Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:2786-2799 bool UseInitArrayDefault = getTriple().getArch() == llvm::Triple::aarch64 || getTriple().getArch() == llvm::Triple::aarch64_be ||

[PATCH] D71393: Default to -fuse-init-array

2019-12-12 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGb2b5cac3ec07: Default to -fuse-init-array (authored by MaskRay). Changed prior to commit: https://reviews.llvm.org/D71393?vs=233503&id=233655#toc Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D71393: Default to -fuse-init-array

2019-12-12 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. +1, the list of exceptions that will use .ctors is smaller than the list of platforms that default to init_array. And in general, I support anything that reduces the length of -cc1 command lines for