<div dir="ltr">On Fri, Aug 16, 2013 at 2:56 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im">On Fri, Aug 16, 2013 at 11:40 AM, Kaylor, Andrew <span dir="ltr"><<a href="mailto:andrew.kaylor@intel.com" target="_blank">andrew.kaylor@intel.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Hi Eli,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Thanks for the suggestions regarding documentation.  I’ll see what I can put together.  I believe I still have the diagrams you referred to, and I can put together
 something more to describe the control flow.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">BTW, I wanted to let you know that your old blog posts describing the PIC relocation model were extremely helpful as I was putting this patch together.  Yours
 was the clearest description of PIC handling that I was able to find.</span></p></div></div></blockquote><div><br></div></div><div>Glad it helped! I'll have to reread them myself before I review your patch ;-)</div>
<div class="im"><div> </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u><u></u></span></p>



<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Also, I meant to ask what ever happened to the lit subtest patch that you submitted about a year and a half ago.  As far as I can tell it was never committed,
 but I couldn’t tell why not.  The “new” tests I’m introducing are just duplications of other tests with variations in invocation arguments.  It would be nice to be able to simplify that ExecutionEngine testing tree.</span></p>


</div></blockquote><div><br></div></div><div>I recall the patch was very close to being done, and Daniel Dunbar was OK with it, but somehow in the last stages of reviewing we both got distracted. The real problem is that the last versions of the patch were mailed directly to Daniel from... my old @<a href="http://intel.com" target="_blank">intel.com</a> address. So unless Daniel (CCd) still has them lying somewhere, I fear the patch is lost :-/ Not that it was a big deal, and it would have to be tweaked in light of the recent additions to FileCheck anyhow. I still believe it's a good thing to have and will be happy to help reviewing a new patch if anyone is willing to take it over.</div>
</div></div></div></blockquote><div><br></div><div>I've attached the last version I got from you. I think if we were going to revisit adding this I might want to take a slightly different approach in light of some other directions I would like to take lit. Primarily I would like to consider lifting the subtest handling up to the generic lit layer instead of specific to the ShTest format. There are other test formats I am hoping to develop that could make use of some kind of explicit subtest support. This would also solve one of the outstanding issues with the patch which was how to compute an overall test result status in the presence of XFAILs, XPASS, etc.</div>
<div><br></div><div>I would also like to know how support for parameterized tests overlaps with this. For MCJIT specifically it sounds like what is really desired is some support for parameterized tests so that all the JIT tests can be run with and without the MCJIT. That feature has come up in the past as well, and could be really useful in cases where we want to, e.g., run a test for each configured target.</div>
<div><br></div><div> - Daniel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="HOEnZb"><font color="#888888">

<div><br></div><div>Eli</div><div><br></div><div><br></div><div><br></div><div> </div></font></span></div></div></div>
</blockquote></div><br></div></div>