>> +
>> + // 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.
{}
// 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