> Test created and new patch attached. I'm a bit uncertain about testing
> the "-analyzer-checker-help" option since that is within #ifdef
> CLANG_ENABLE_STATIC_ANALYZER in the code. Will this flag always be
> active when running the tests? If not how should I check this in the
> test?

It is probably best to leave it out. The patch LGTM with that changed.

> Perhaps I should mention that I don't have commit access so I will
> need help with that part later.

I will commit. Sorry for taking so long on the review.

Cheers,
Rafael
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to