Fix constructor ordering

Author: sethrj

src/qiree/Module.cc

@@ -118,6 +118,7 @@ Module::Module(UPModule&& module) : module_{std::move(module)}
 //---------------------------------------------------------------------------//
 /*!
  * Construct with an LLVM module and an entry point.
+ *
  * Useful when there are multiple entry points.
  */
 Module::Module(UPModule&& module, std::string const& entrypoint)
@@ -158,14 +159,8 @@ Module::Module(std::string const& filename, std::string const& entrypoint)
 
 //---------------------------------------------------------------------------//
 /*!
- * Reading a module by parsing an in-memory LLVM IR string.
+ * Read a module by parsing an in-memory LLVM IR string.
  */
-
-Module::Module() = default;
-Module::~Module() = default;
-Module::Module(Module&&) = default;
-Module& Module::operator=(Module&&) = default;
-
 std::unique_ptr<Module> Module::from_bytes(std::string const& content)
 {
     llvm::SMDiagnostic err;
@@ -189,6 +184,14 @@ std::unique_ptr<Module> Module::from_bytes(std::string const& content)
     return std::make_unique<Module>(std::move(llvm_module));
 }
 
+//! Construct in an empty state
+Module::Module() = default;
+
+// Default destructor and move
+Module::~Module() = default;
+Module::Module(Module&&) = default;
+Module& Module::operator=(Module&&) = default;
+
 //---------------------------------------------------------------------------//
 /*!
  * Process entry point attributes.

src/qiree/Module.hh

@@ -33,16 +33,6 @@ class Module
     //!@}
 
   public:
-    // Default empty constructor
-    Module();
-    // Externally defined defaults
-    ~Module();
-    Module(Module&&);
-    Module& operator=(Module&&);
-    // Prevent copying
-    Module(Module const&) = delete;
-    Module& operator=(Module const&) = delete;
-
     // Reading a module by parsing an in-memory LLVM IR string.
     static std::unique_ptr<Module> from_bytes(std::string const & content);
 
@@ -58,6 +48,20 @@ class Module
     // Construct with an LLVM IR file (bitcode or disassembled) and entry point
     Module(std::string const& filename, std::string const& entrypoint);
 
+    // Construct in an empty state
+    Module();
+
+    // Externally defined defaults
+    ~Module();
+    Module(Module&&);
+    Module& operator=(Module&&);
+
+    // Prevent copying
+    Module(Module const&) = delete;
+    Module& operator=(Module const&) = delete;
+
+    //// ACCESSORS ////
+
     // Process entry point attributes
     EntryPointAttrs load_entry_point_attrs() const;