<div dir="ltr">Works for me.<br><br>Thanks!<div><br></div><div>-eric</div></div><br><div class="gmail_quote">On Sun, Mar 8, 2015 at 4:32 AM Ed Schouten <<a href="mailto:ed@nuxi.nl">ed@nuxi.nl</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2015-03-07 17:46 GMT+01:00 Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>>:<br>
> That's fine. Just reply with the patch with testcase for the record. :)<br>
<br>
Sure!<br>
<br>
Before Eric pointed me to the unit tests for Triple, I was under the<br>
impression that we only tested triples by having accidental testing<br>
coverage for them through the IR dumps we have in test/. My idea was<br>
therefore to at least add some coverage for CloudABI by adjusting one<br>
of the existing tests to use the CloudABI triple:<br>
<br>
Index: test/CodeGen/X86/dwarf-eh-<u></u>prepare.ll<br>
==============================<u></u>==============================<u></u>=======<br>
--- test/CodeGen/X86/dwarf-eh-<u></u>prepare.ll        (revision 231492)<br>
+++ test/CodeGen/X86/dwarf-eh-<u></u>prepare.ll        (working copy)<br>
@@ -1,4 +1,4 @@<br>
-; RUN: opt -mtriple=x86_64-linux-gnu -dwarfehprepare < %s -S | FileCheck %s<br>
+; RUN: opt -mtriple=x86_64-unknown-<u></u>cloudabi -dwarfehprepare < %s -S |<br>
FileCheck %s<br>
<br>
 ; Check basic functionality of IR-to-IR DWARF EH preparation. This should<br>
 ; eliminate resumes. This pass requires a TargetMachine, so we put it under X86<br>
<br>
Further testing (after I sent the patch to Eric) revealed that this<br>
change makes little sense. LLVM doesn't really care if the triple you<br>
provide is valid at all. I used a couple of random OS names but none<br>
of them caused the test to fail.<br>
<br>
My proposal is to stick to the original patch I sent out two days ago,<br>
including the change to unittests/ADT/TripleTest.cpp. Attached is a<br>
combined patch.<br>
<br>
--<br>
Ed Schouten <<a href="mailto:ed@nuxi.nl" target="_blank">ed@nuxi.nl</a>><br>
</blockquote></div>