List Subzero's local optlevel flags after LLVM's cxxflags (precedence).

Ended up needing to fix the InstX8632Lockable error that JF ran into now that -O0 is really -O0: https://codereview.chromium.org/512933006/ BUG=none R=stichnot@chromium.org Review URL: https://codereview.chromium.org/557933002

List Subzero's local optlevel flags after LLVM's cxxflags (precedence).
a262333e · Jan Voung · 6d4f5647 · a262333e · a262333e
Commit a262333e authored Sep 09, 2014 by Jan Voung
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 6 deletions

Makefile.standalone Makefile.standalone +2 -2

IceInstX8632.h src/IceInstX8632.h +0 -4

No files found.
--- a/Makefile.standalone
+++ b/Makefile.standalone
@@ -41,8 +41,8 @@ LLVM_LDFLAGS := `$(LLVM_BIN_PATH)/llvm-config --libs` \
 OPTLEVEL := -O0
 CXX := g++

-CXXFLAGS := -Wall -Wextra -Werror -fno-rtti -fno-exceptions \
-	$(OPTLEVEL) -g $(LLVM_CXXFLAGS) $(HOST_FLAGS) \
+CXXFLAGS := $(LLVM_CXXFLAGS) -Wall -Wextra -Werror -fno-rtti \
+	-fno-exceptions $(OPTLEVEL) -g $(HOST_FLAGS) \
 	-Wno-error=unused-parameter
 LDFLAGS := $(HOST_FLAGS)


--- a/src/IceInstX8632.h
+++ b/src/IceInstX8632.h
@@ -674,10 +674,6 @@ typedef InstX8632ThreeAddressop<InstX8632::Pshufd> InstX8632Pshufd;

 // Base class for a lockable x86-32 instruction (emits a locked prefix).
 class InstX8632Lockable : public InstX8632 {
-public:
-  virtual void emit(const Cfg *Func) const = 0;
-  virtual void dump(const Cfg *Func) const;
-
 protected:
  bool Locked;