[PATCH] D11075: LLVM gen correct asm info for mingw and cygwin arm targets

Tim Northover t.p.northover at gmail.com
Thu Jul 9 15:38:53 PDT 2015


t.p.northover added a comment.

> I could recreate all the tests with the windows-gnu triplet but they would all fail until http://reviews.llvm.org/D11071 is approved and merged


Speaking of which, that could do with some tests too.

But I'm not sure how a Clang patch can have any influence on LLVM tests. This appears to be changing **something** about how llc would emit object files (and assembly? I don't know) when using these triples. That's what should be tested with this patch.

> I could recreate all the tests with the windows-gnu triplet


If the same behaviour is expected, you should be able to just add a second "RUN" line to those tests.

Cheers.

Tim.


http://reviews.llvm.org/D11075







More information about the cfe-commits mailing list