The patch itself I think is good, but I don't think you need a driver test. Can't you use %clang_cc1 -verify to test?
On 1 August 2012 12:32, Hans Wennborg <[email protected]> wrote: > Hi all, > > The attached patch changes the TargetInfo for OpenBSD to consider TLS > unsupported. (PR13502) > > I'm not sure what the best way to test this is. I've added an attempt, > but would love to get comments here. > > It also makes me wonder if we get this right for other targets. For > instance, what's the status of TLS for NetBSD? Solaris? And I thought > LLVM did support TLS on Windows? > > Please take a look. > > Thanks, > Hans > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
