[PATCH] D76079: [Hexagon] Enable init_arrays when target is linux-musl

2020-03-12 Thread Sid Manning via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG6174fddbe3d4: [Hexagon] Enable init_arrays when target is linux-musl (authored by sidneym). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76079/new/

[PATCH] D76079: [Hexagon] Enable init_arrays when target is linux-musl

2020-03-12 Thread Sid Manning via Phabricator via cfe-commits
sidneym updated this revision to Diff 24. sidneym added a comment. Drop ternary. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76079/new/ https://reviews.llvm.org/D76079 Files: clang/lib/Driver/ToolChains/Hexagon.cpp clang/test/Driver/hexagon-toolchain-elf.c Index:

[PATCH] D76079: [Hexagon] Enable init_arrays when target is linux-musl

2020-03-12 Thread Brian Cain via Phabricator via cfe-commits
bcain added inline comments. Comment at: clang/lib/Driver/ToolChains/Hexagon.cpp:548 + + bool UseInitArrayDefault = (getTriple().isMusl()) ? true : false; The ternary is unnecessary. ``` bool UseInitArrayDefault = getTriple().isMusl(); ``` Repository:

[PATCH] D76079: [Hexagon] Enable init_arrays when target is linux-musl

2020-03-12 Thread Sid Manning via Phabricator via cfe-commits
sidneym created this revision. sidneym added reviewers: kparzysz, bcain, adasgupt. Herald added a project: clang. Herald added a subscriber: cfe-commits. linux/musl will use init arrays by default. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D76079 Files: