diff options
| author | Jamie Madill <jmadill@chromium.org> | 2022-07-29 21:26:37 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-07-29 21:26:37 +0300 |
| commit | a90ccc240501bf3362b23f67771f65b7dec2ccf9 (patch) | |
| tree | 7c68532454b058cac20ee2d4cfd7b443686d2309 | |
| parent | 4773879b6846930089fd9c2520662933e65e25ad (diff) | |
Remove default copy constructor in header. (#4879)
A recent libc++ roll in Chrome warned of a deprecated copy. We're
still looking if this is a bug in libc++ or a valid warning, but
removing the redundant line is a safe workaround or fix in either
case.
See discussion in https://crrev.com/c/3791771
| -rw-r--r-- | source/opt/merge_return_pass.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/source/opt/merge_return_pass.h b/source/opt/merge_return_pass.h index a35cf269f..d15db2f67 100644 --- a/source/opt/merge_return_pass.h +++ b/source/opt/merge_return_pass.h @@ -118,8 +118,6 @@ class MergeReturnPass : public MemPass { StructuredControlState(Instruction* break_merge, Instruction* merge) : break_merge_(break_merge), current_merge_(merge) {} - StructuredControlState(const StructuredControlState&) = default; - bool InBreakable() const { return break_merge_; } bool InStructuredFlow() const { return CurrentMergeId() != 0; } |