search for: addanalysisusage

I've just had some fun, because I wanted to override FunctionPass::addAnalysisUsage, but forgot "const" after the method name -- so instead of overriding I've just created a new unrelated method. After spending some time on this, I've decided to add -Woverloaded-virtual option to compiler to catch such cases. However, it also gives some warnings on LLVM code:...

On Thu, 24 Jun 2004, Vladimir Prus wrote: > I've just had some fun, because I wanted to override > FunctionPass::addAnalysisUsage, but forgot "const" after the method name -- > so instead of overriding I've just created a new unrelated method. Ya know, I think that everyone has done that at least once (myself included)... on THAT VERY METHOD. grr :) I definitely think that it would be a good idea to add th...

...we need to file a bug to take care of the warnings it produces so that LLVm is overloaded virtual clean, then we can turn on checking for overloaded virtuals. Reid. On Thu, 2004-06-24 at 06:40, Vladimir Prus wrote: > I've just had some fun, because I wanted to override > FunctionPass::addAnalysisUsage, but forgot "const" after the method name -- > so instead of overriding I've just created a new unrelated method. > > After spending some time on this, I've decided to add -Woverloaded-virtual > option to compiler to catch such cases. However, it also gives some war...

...gt; so that LLVm is overloaded virtual clean, then we can turn on checking > for overloaded virtuals. This sounds like a great way to do it. -Chris > On Thu, 2004-06-24 at 06:40, Vladimir Prus wrote: > > I've just had some fun, because I wanted to override > > FunctionPass::addAnalysisUsage, but forgot "const" after the method name -- > > so instead of overriding I've just created a new unrelated method. > > > > After spending some time on this, I've decided to add -Woverloaded-virtual > > option to compiler to catch such cases. However, it al...