[llvm-commits] [llvm] r96282 - in /llvm/trunk: lib/VMCore/Verifier.cpp unittests/VMCore/VerifierTest.cpp

[Chris Lattner]

On Feb 16, 2010, at 1:23 PM, David Greene wrote:

> On Tuesday 16 February 2010 00:36:23 Chris Lattner wrote:
> one of those testing methodology types... ;]
>> 
>> I don't like it (if it creeps beyond this) because it adds more uses of the
>> C++ API, meaning that the tests need to be updated when the API changes. 
>> Why not use the stable C api for stuff like this?
> 
> On the other hand, tests like this encourage people to think more about
> API changes and what is really necessary.  I don't see that as a bad thing.
> It won't prevent changes but might encourage people to think more about
> what they're doing and/or provide backward-compatibility if appropriate.

I completely disagree, we are not interesting in avoiding api changes because of inconvenience.

-Chris