[llvm] [SPIRV] Fix code quality issues. (PR #152005)
Nathan Gauër via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 5 04:16:52 PDT 2025
================
@@ -101,6 +106,9 @@ class ConvergenceRegionInfo {
~ConvergenceRegionInfo() { releaseMemory(); }
+ ConvergenceRegionInfo(const ConvergenceRegionInfo &LHS) = default;
+ ConvergenceRegionInfo &operator=(const ConvergenceRegionInfo &LHS) = default;
----------------
Keenuts wrote:
I don't think we want this. Since the move ctor has been declared, the copy ctor has been implicitly deleted. Which is IMO a good thing since we don't want to have copies of the TopLevelRegion.
Now we can make it explicit by marking both as `delete`.
https://github.com/llvm/llvm-project/pull/152005
More information about the llvm-commits
mailing list