[cfe-dev] patch: adding an assert to EvalLoad() in the Clang static analyzer

Ted Kremenek kremenek at apple.com
Tue Nov 23 17:08:47 PST 2010


Please go ahead an add the assert.
On Nov 22, 2010, at 10:37 PM, Zhanyong Wan (λx.x x) wrote:

> Hi Ted,
> 
> While fixing PR8419, I added an assert to catch errors in using
> EvalLoad().  I didn't commit it as part of the fix, as I figured that
> you prefer that each patch has a single purpose.  I've created a
> separate patch for the assert() and uploaded it to
> http://codereview.appspot.com/3272041 -- would you please review it?
> Thanks,
> 
> -- 
> Zhanyong





More information about the cfe-dev mailing list