[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