Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(402)

Side by Side Diff: src/IceCompileServer.cpp

Issue 1197863003: Subzero: Reduce the amount of #ifdef'd code. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Cleanup Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/IceClFlags.cpp ('k') | src/IceCompiler.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 //===- subzero/src/IceCompileServer.cpp - Compile server ------------------===// 1 //===- subzero/src/IceCompileServer.cpp - Compile server ------------------===//
2 // 2 //
3 // The Subzero Code Generator 3 // The Subzero Code Generator
4 // 4 //
5 // This file is distributed under the University of Illinois Open Source 5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details. 6 // License. See LICENSE.TXT for details.
7 // 7 //
8 //===----------------------------------------------------------------------===// 8 //===----------------------------------------------------------------------===//
9 // 9 //
10 // This file defines the basic commandline-based compile server. 10 // This file defines the basic commandline-based compile server.
(...skipping 17 matching lines...) Expand all
28 #include "IceClFlags.h" 28 #include "IceClFlags.h"
29 #include "IceClFlagsExtra.h" 29 #include "IceClFlagsExtra.h"
30 #include "IceCompileServer.h" 30 #include "IceCompileServer.h"
31 #include "IceELFStreamer.h" 31 #include "IceELFStreamer.h"
32 #include "IceGlobalContext.h" 32 #include "IceGlobalContext.h"
33 33
34 namespace Ice { 34 namespace Ice {
35 35
36 namespace { 36 namespace {
37 37
38 static_assert( 38 static_assert(!(BuildDefs::textualBitcode() && PNACL_BROWSER_TRANSLATOR),
39 !(INPUT_IS_TEXTUAL_BITCODE && PNACL_BROWSER_TRANSLATOR), 39 "Can not define INPUT_IS_TEXTUAL_BITCODE when building browswer "
40 "Can not define INPUT_IS_TEXTUAL_BITCODE when building browswer translator"); 40 "translator");
41 41
42 // Define a SmallVector backed buffer as a data stream, so that it 42 // Define a SmallVector backed buffer as a data stream, so that it
43 // can hold the generated binary version of the textual bitcode in the 43 // can hold the generated binary version of the textual bitcode in the
44 // input file. 44 // input file.
45 class TextDataStreamer : public llvm::DataStreamer { 45 class TextDataStreamer : public llvm::DataStreamer {
46 public: 46 public:
47 TextDataStreamer() = default; 47 TextDataStreamer() = default;
48 ~TextDataStreamer() final = default; 48 ~TextDataStreamer() final = default;
49 static TextDataStreamer *create(const IceString &Filename, std::string *Err); 49 static TextDataStreamer *create(const IceString &Filename, std::string *Err);
50 size_t GetBytes(unsigned char *Buf, size_t Len) final; 50 size_t GetBytes(unsigned char *Buf, size_t Len) final;
51
51 private: 52 private:
52 llvm::SmallVector<char, 1024> BitcodeBuffer; 53 llvm::SmallVector<char, 1024> BitcodeBuffer;
53 size_t Cursor = 0; 54 size_t Cursor = 0;
54 }; 55 };
55 56
56 TextDataStreamer *TextDataStreamer::create(const IceString &Filename, 57 TextDataStreamer *TextDataStreamer::create(const IceString &Filename,
57 std::string *Err) { 58 std::string *Err) {
58 TextDataStreamer *Streamer = new TextDataStreamer(); 59 TextDataStreamer *Streamer = new TextDataStreamer();
59 llvm::raw_string_ostream ErrStrm(*Err); 60 llvm::raw_string_ostream ErrStrm(*Err);
60 if (std::error_code EC = llvm::readNaClRecordTextAndBuildBitcode( 61 if (std::error_code EC = llvm::readNaClRecordTextAndBuildBitcode(
(...skipping 30 matching lines...) Expand all
91 92
92 ErrorCodes getReturnValue(const Ice::ClFlagsExtra &Flags, ErrorCodes Val) { 93 ErrorCodes getReturnValue(const Ice::ClFlagsExtra &Flags, ErrorCodes Val) {
93 if (Flags.getAlwaysExitSuccess()) 94 if (Flags.getAlwaysExitSuccess())
94 return EC_None; 95 return EC_None;
95 return Val; 96 return Val;
96 } 97 }
97 98
98 } // end of anonymous namespace 99 } // end of anonymous namespace
99 100
100 void CLCompileServer::run() { 101 void CLCompileServer::run() {
101 if (ALLOW_DUMP) { 102 if (BuildDefs::dump()) {
102 llvm::sys::PrintStackTraceOnErrorSignal(); 103 llvm::sys::PrintStackTraceOnErrorSignal();
103 } 104 }
104 ClFlags::parseFlags(argc, argv); 105 ClFlags::parseFlags(argc, argv);
105 ClFlags Flags; 106 ClFlags Flags;
106 ClFlagsExtra ExtraFlags; 107 ClFlagsExtra ExtraFlags;
107 ClFlags::getParsedClFlags(Flags); 108 ClFlags::getParsedClFlags(Flags);
108 ClFlags::getParsedClFlagsExtra(ExtraFlags); 109 ClFlags::getParsedClFlagsExtra(ExtraFlags);
109 110
110 std::error_code EC; 111 std::error_code EC;
111 std::unique_ptr<Ostream> Ls = makeStream(ExtraFlags.getLogFilename(), EC); 112 std::unique_ptr<Ostream> Ls = makeStream(ExtraFlags.getLogFilename(), EC);
(...skipping 29 matching lines...) Expand all
141 *Ls << "Failed to open output file: " << ExtraFlags.getOutputFilename() 142 *Ls << "Failed to open output file: " << ExtraFlags.getOutputFilename()
142 << ":\n" << EC.message() << "\n"; 143 << ":\n" << EC.message() << "\n";
143 return transferErrorCode(getReturnValue(ExtraFlags, Ice::EC_Args)); 144 return transferErrorCode(getReturnValue(ExtraFlags, Ice::EC_Args));
144 } 145 }
145 Os->SetUnbuffered(); 146 Os->SetUnbuffered();
146 } break; 147 } break;
147 } 148 }
148 149
149 IceString StrError; 150 IceString StrError;
150 std::unique_ptr<llvm::DataStreamer> InputStream( 151 std::unique_ptr<llvm::DataStreamer> InputStream(
151 INPUT_IS_TEXTUAL_BITCODE 152 BuildDefs::textualBitcode()
152 ? TextDataStreamer::create(ExtraFlags.getIRFilename(), &StrError) 153 ? TextDataStreamer::create(ExtraFlags.getIRFilename(), &StrError)
153 : llvm::getDataFileStreamer(ExtraFlags.getIRFilename(), &StrError) 154 : llvm::getDataFileStreamer(ExtraFlags.getIRFilename(), &StrError));
154 );
155 if (!StrError.empty() || !InputStream) { 155 if (!StrError.empty() || !InputStream) {
156 llvm::SMDiagnostic Err(ExtraFlags.getIRFilename(), 156 llvm::SMDiagnostic Err(ExtraFlags.getIRFilename(),
157 llvm::SourceMgr::DK_Error, StrError); 157 llvm::SourceMgr::DK_Error, StrError);
158 Err.print(ExtraFlags.getAppName().c_str(), *Ls); 158 Err.print(ExtraFlags.getAppName().c_str(), *Ls);
159 return transferErrorCode(getReturnValue(ExtraFlags, Ice::EC_Bitcode)); 159 return transferErrorCode(getReturnValue(ExtraFlags, Ice::EC_Bitcode));
160 } 160 }
161 161
162 Ctx.reset( 162 Ctx.reset(
163 new GlobalContext(Ls.get(), Os.get(), Ls.get(), ELFStr.get(), Flags)); 163 new GlobalContext(Ls.get(), Os.get(), Ls.get(), ELFStr.get(), Flags));
164 if (Ctx->getFlags().getNumTranslationThreads() != 0) { 164 if (Ctx->getFlags().getNumTranslationThreads() != 0) {
165 std::thread CompileThread([this, &ExtraFlags, &InputStream]() { 165 std::thread CompileThread([this, &ExtraFlags, &InputStream]() {
166 Ctx->initParserThread(); 166 Ctx->initParserThread();
167 getCompiler().run(ExtraFlags, *Ctx.get(), std::move(InputStream)); 167 getCompiler().run(ExtraFlags, *Ctx.get(), std::move(InputStream));
168 }); 168 });
169 CompileThread.join(); 169 CompileThread.join();
170 } else { 170 } else {
171 getCompiler().run(ExtraFlags, *Ctx.get(), std::move(InputStream)); 171 getCompiler().run(ExtraFlags, *Ctx.get(), std::move(InputStream));
172 } 172 }
173 transferErrorCode(getReturnValue( 173 transferErrorCode(getReturnValue(
174 ExtraFlags, static_cast<ErrorCodes>(Ctx->getErrorStatus()->value()))); 174 ExtraFlags, static_cast<ErrorCodes>(Ctx->getErrorStatus()->value())));
175 } 175 }
176 176
177 } // end of namespace Ice 177 } // end of namespace Ice
OLDNEW
« no previous file with comments | « src/IceClFlags.cpp ('k') | src/IceCompiler.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698