[llvm] 588ffda - [polly] Fix compiler warning. NFC.

[Michael Kruse via llvm-commits]

Am Di., 17. Mai 2022 um 12:56 Uhr schrieb David Blaikie <dblaikie at gmail.com>:
> Could the dtor be non-virtual but protected in DOTGraphTraits* (if
> these types are not instantiated directly) and then the derived
> classes marked as final? Just a thought, if the virtual destruction
> isn't actually needed, that's one way to express the API intent more
> accurately than adding an unneeded virtual.

That actually does avoid the warning, although I do not understand why
it is not necessary to redeclare a public dtor in all the derived
classes. Is the implicit one always public?

Review here: reviews.llvm.org/D125830

Michael

-- 
Tardyzentrismus verboten!