[cfe-commits] r146959 - in /cfe/trunk: include/clang/AST/ include/clang/Analysis/Analyses/ include/clang/Basic/ include/clang/CodeGen/ include/clang/Driver/ include/clang/Frontend/ include/clang/Lex/ include/clang/Sema/ include/clang/StaticAnalyzer/Core/ include/clang/StaticAnalyzer/Core/BugReporter/ include/clang/StaticAnalyzer/Core/PathSensitive/ lib/AST/ lib/Analysis/ lib/Basic/ lib/CodeGen/ lib/Driver/ lib/Frontend/ lib/Lex/ lib/Parse/ lib/Sema/ lib/StaticAnalyzer/Core/ utils/TableGen/

[Rafael Ávila de Espíndola]

> If there is a simple virtual method that can be moved out of line
> without impacting performance, then that is clearly preferable to adding
> an explicit anchor method.  The main advantage of this patch IMO is that
> we could turn on the warning flag in the build to catch future cases
> that should have an anchor.

Ah, I had missed the bit about turning on the waring on our build. I
agree it is a small price to pay for using the shortcut.

> -Chris
> 

Cheers,
Rafael