diff options
| author | Jeff Law <jlaw@ventanamicro.com> | 2025-09-07 12:28:23 -0600 |
|---|---|---|
| committer | Jeff Law <jlaw@ventanamicro.com> | 2025-09-07 12:28:23 -0600 |
| commit | 1b9c218d1429d596a33b13e450da0bdc1643e984 (patch) | |
| tree | 91918f1620245c7fa0f194ae31851f2e792fdabe | |
| parent | c4d5383b1d5771bf81659103f4f67829d791067d (diff) | |
| download | gcc-1b9c218d1429d596a33b13e450da0bdc1643e984.zip gcc-1b9c218d1429d596a33b13e450da0bdc1643e984.tar.gz gcc-1b9c218d1429d596a33b13e450da0bdc1643e984.tar.bz2 | |
gcc: introduce the dep_fusion pass
>> +
>> + // opt_pass methods:
>> + opt_pass *clone () override { return new pass_dep_fusion (m_ctxt); }
>> + bool gate (function *) override;
>> + unsigned int execute (function *) override;
>
> Wouldn't it be better to add 'final' along with 'override' to opt_pass
> vfuncs?
> (See commit 725793af78064fa605ea6d9376aaf99ecb71467b, etc.)Yea. It's easily missed. Fixed in the obvious way.
Bootstrapped and regression tested on x86_64. Pushed to the trunk.
gcc/
* dep-fusion.cc: Mark clone, gate and execute methods as final.
| -rw-r--r-- | gcc/dep-fusion.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gcc/dep-fusion.cc b/gcc/dep-fusion.cc index d040af6..bbd9351 100644 --- a/gcc/dep-fusion.cc +++ b/gcc/dep-fusion.cc @@ -61,9 +61,9 @@ public: {} // opt_pass methods: - opt_pass *clone () override { return new pass_dep_fusion (m_ctxt); } - bool gate (function *) override; - unsigned int execute (function *) override; + opt_pass *clone () final override { return new pass_dep_fusion (m_ctxt); } + bool gate (function *) final override; + unsigned int execute (function *) final override; }; bool |