[llvm-branch-commits] [lld] [NFC][ELF] Remove Symbol's unused copy constructor (PR #210615)

[Jessica Clarke via llvm-branch-commits]

================
@@ -78,9 +78,17 @@ class Symbol {
   // The file from which this symbol was created.
   InputFile *file;
 
-  // The default copy constructor is deleted due to atomic flags. Define one for
-  // places where no atomic is needed.
-  Symbol(const Symbol &o) { memcpy(static_cast<void *>(this), &o, sizeof(o)); }
+  // Although the copy/move constructors/assignment operators are deleted due
----------------
jrtc27 wrote:

Thanks, agreed that's a better comment, and I have now learnt something new about C++. Just test building then will push with that applied.

https://github.com/llvm/llvm-project/pull/210615