[patch] Check the linkage cache at every recursive step
Rafael Espíndola
rafael.espindola at gmail.com
Tue May 28 11:37:27 PDT 2013
> Well, wait, associated question — does this actually cause a speed-up?
For SemaExpr.ii it is in the noise, which is why I decided to count
the number of linkage computations.
The patch has some other advantages:
* verifyLinkage is gone (inlined to getLVForDecl)
* When asserts are enabled, we check the linkage even when it is
computed as part of a recursive getLVForDecl. That is how I found the
bug fixed in r182773.
OK?
Cheers,
Rafael
More information about the cfe-commits
mailing list