[llvm] r262790 - unitests: add some ARM TargetParser tests

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 8 20:56:29 PST 2016


On 8 March 2016 at 12:06, Saleem Abdulrasool <compnerd at compnerd.org> wrote:
> I was actually half tempted to drop a clang test that I added (which then
> prompted me to actually put the effort in for this work).  Im torn on
> whether that is a good thing or not since that ensures that the frontend
> actually emits an error, which is a good thing to check anyways.

Indeed, we need to keep some tests in Clang, to make sure the Clang
driver is using the API correctly, but we can easily move 90% of those
tests to a unit test in LLVM.

cheers,
--renato


More information about the llvm-commits mailing list