LGTM, please commit. (I wouldn't mind it if there were some sense that this was a decision made by the Solaris platform - perhaps you are such a spokesperson for Solaris, but I don't know)
On Wed, Oct 15, 2014 at 10:10 AM, Alexander Eremin < [email protected]> wrote: > Thanks for pointing, this is updated patch. > > Alex > > 2014-10-15 19:24 GMT+04:00 David Blaikie <[email protected]>: > >> Sorry, I'm not sure I follow - what I meant is the patch needs (or should >> have, unless there's a reason this isn't testable, etc) a test case in >> Clang's regression suite. Probably in clang/test/CodeGen/debug-info-* >> >> (you could try removing other targets from that conditional and see which >> tests fail - that should piont you to where to add tests for this platform) >> >> > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
