[PATCH] D151699: [VLIWMachineScheduler] Disable default copy ctor and copy assignment operator for VLIWSchedBoundary

[Bing Yu via Phabricator via llvm-commits]

yubing added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/VLIWMachineScheduler.h:155
+    VLIWSchedBoundary &operator=(const VLIWSchedBoundary &other) = delete;
+    VLIWSchedBoundary(const VLIWSchedBoundary &other) = delete;
 
----------------
JamesNagurne wrote:
> What is the reasoning here? It's not obvious to me.
class VLIWSchedBoundary manages resources such as dynamically allocated memory, it's generally a good practice to either implement a custom copy constructor or disable the default one.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151699/new/

https://reviews.llvm.org/D151699