[PATCH] [CFI] Require -flto instead of implying it.

Peter Collingbourne peter at pcc.me.uk
Thu Jun 18 18:46:23 PDT 2015


LGTM

I think you will also need to adjust `config.lto_flags` in `compiler-rt/test/lit.common.cfg`.


================
Comment at: lib/Driver/Driver.cpp:1387
@@ -1390,2 +1386,3 @@
+bool Driver::IsUsingLTO(const ArgList &Args) const {
   if (Args.hasFlag(options::OPT_flto, options::OPT_fno_lto, false))
     return true;
----------------
This could just be `return Args.hasFlag(...)` now.

http://reviews.llvm.org/D10560

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list