From: Daniel Dunbar <daniel@zuster.org>
Date: Mon, 30 Nov 2009 08:41:13 +0000 (+0000)
Subject: clang -cc1: Initialize LangOptions::{Optimize,NoInline}
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1c5944c77383b1f3cbf33ea9c656880c0bd42eff;p=clang

clang -cc1: Initialize LangOptions::{Optimize,NoInline}

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@90123 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Driver/CC1Options.cpp b/lib/Driver/CC1Options.cpp
index 88cbd5491d..667335e0b4 100644
--- a/lib/Driver/CC1Options.cpp
+++ b/lib/Driver/CC1Options.cpp
@@ -579,8 +579,18 @@ static void ParseLangArgs(LangOptions &Opts, ArgList &Args,
   Opts.PICLevel = getLastArgIntValue(Args, OPT_pic_level, 0, Diags);
   Opts.Static = Args.hasArg(OPT_static_define);
   Opts.OptimizeSize = 0;
-  Opts.Optimize = 0; // FIXME!
-  Opts.NoInline = 0; // FIXME!
+
+  // FIXME: Eliminate this dependency.
+  unsigned Opt =
+    Args.hasArg(OPT_Os) ? 2 : getLastArgIntValue(Args, OPT_O, 0, Diags);
+  Opts.Optimize = Opt != 0;
+
+  // This is the __NO_INLINE__ define, which just depends on things like the
+  // optimization level and -fno-inline, not actually whether the backend has
+  // inlining enabled.
+  //
+  // FIXME: This is affected by other options (-fno-inline).
+  Opts.NoInline = !Opt;
 
   unsigned SSP = getLastArgIntValue(Args, OPT_stack_protector, 0, Diags);
   switch (SSP) {