Includes module header first.

src/IceBrowserCompileServer.cpp

 //===- subzero/src/IceBrowserCompileServer.cpp - Browser compile server ---===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file defines the browser-based compile server.
 //
 //===----------------------------------------------------------------------===//
 
 // Can only compile this with the NaCl compiler (needs irt.h, and the
 // unsandboxed LLVM build using the trusted compiler does not have irt.h).
 #if PNACL_BROWSER_TRANSLATOR
 
+#include "IceBrowserCompileServer.h"
+
 #include <cstring>

[Jim Stichnoth, 2015/06/23 23:05:04]

I just became aware of http://llvm.org/docs/CodingStandards.html#include-style .
 What do you think about making Subzero conformant by moving all the
<system_includes> to the end?

[John, 2015/06/23 23:57:09]

but that means goig through EVERY subzero file... :(

Oh, well, done. :)

 #include <irt.h>
 #include <irt_dev.h>
 #include <pthread.h>
 #include <thread>
 
 #include "llvm/Support/QueueStreamer.h"
 
-#include "IceBrowserCompileServer.h"
 
 namespace Ice {
 
 // Create C wrappers around callback handlers for the IRT interface.
 namespace {
 
 BrowserCompileServer *gCompileServer;
 struct nacl_irt_private_pnacl_translator_compile gIRTFuncs;
 
 void getIRTInterfaces() {
@@ skipping 152 matching lines @@
     this->getCompiler().run(ExtraFlags, *Ctx.get(),
                             // Retain original reference, but the compiler
                             // (LLVM's MemoryObject) wants to handle deletion.
                             std::unique_ptr<llvm::DataStreamer>(InputStream));
   });
 }
 
 } // end of namespace Ice
 
 #endif // PNACL_BROWSER_TRANSLATOR