<div dir="ltr">Might be better to reduce some source code, produce assembly, then tweak that slightly before going to yaml. Not sure.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 5, 2017 at 10:05 AM Jonas Devlieghere via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JDevlieghere added a comment.<br>
<br>
@dblaikie  wrote:<br>
<br>
> Might be worth looking at the previous review on this: <a href="https://reviews.llvm.org/D32821" rel="noreferrer" target="_blank">https://reviews.llvm.org/D32821</a><br>
><br>
> (yeah, it stalled out - partly my fault, I did go back to try to work up the changes I'd been suggesting, but hit some other issues that I gave up on - might be worth discussing design first a bit before diving into the code changes)<br>
><br>
> I imagine a test for this could be reduced beyond 700 lines of YAML - happy to help with that if you get stuck/have trouble reducing/simplifying it.<br>
><br>
> (I mean there's several things to test for here, but one basic one should be well reduced to ~10s of lines, hopefully)<br>
<br>
<br>
Thanks David, that looks very similar indeed! The truth is that I don't really have an idea how to get started on reducing a YAML test case without breaking everything. I've given it a try but the result wasn't valid anymore, even before changing the ranges.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D37481" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37481</a><br>
<br>
<br>
<br>
</blockquote></div>