[PATCH] D48756: Add option for section ordering

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

This version is ok with me, question below around whether we want 2 separate options here though.

Davide, did you consider changing the default for the clang driver options instead of just in gold/lld for LTO? It might also be nice to hook up -fno-function-sections and -fno-data-sections to this new gold option (and mirror it in lld), just as I had done for the affirmative versions of those options in tools::AddGoldPlugin (which is now essentially a no-op due to the default change). In which case we would want 2 separate options here (one for functions and one for data). Thoughts?


Repository:
  rL LLVM

https://reviews.llvm.org/D48756