Implement move semantics and disable copy semantics for the new PipeReader

gpu-simulator/trace-parser/trace_parser.cc

@@ -473,4 +473,21 @@ bool PipeReader::hasEnding(const std::string &fullString,
                                     ending.length(), ending));
   }
   return false;
+}
+
+PipeReader::PipeReader(PipeReader &&other) noexcept
+    : pipe(other.pipe), command(other.command) {
+  other.pipe = NULL;
+  other.command = {};
+}
+
+PipeReader &PipeReader::operator=(PipeReader &&other) noexcept {
+  if (this != &other) {
+    pipe = other.pipe;
+    command = other.command;
+
+    other.pipe = NULL;
+    other.command = {};
+  }
+  return *this;
 }

gpu-simulator/trace-parser/trace_parser.h

@@ -78,7 +78,6 @@ struct inst_trace_t {
 class PipeReader {
  public:
   PipeReader(const std::string &filePath);
-  void OpenFile(const std::string &filePath);
 
   // Destructor to close the pipe
   ~PipeReader() {
@@ -87,6 +86,15 @@ class PipeReader {
     }
   }
 
+  // It does not make sense to implement copy semantics for PipeReader,
+  // because each instance should hold a unique Linux pipe handle
+  PipeReader(const PipeReader &) = delete;
+  PipeReader &operator=(const PipeReader &) = delete;
+
+  // Move semantics can be supported
+  PipeReader(PipeReader &&) noexcept;
+  PipeReader &operator=(PipeReader &&) noexcept;
+
   // Read one line
   bool readLine(std::string &line);
 
@@ -97,6 +105,8 @@ class PipeReader {
   // Helper function to check if a string ends with a specific suffix (file
   // extension)
   bool hasEnding(const std::string &fullString, const std::string &ending);
+
+  void OpenFile(const std::string &filePath);
 };
 
 struct kernel_trace_t {