
<html>


<head>


<meta http-equiv="Content-Type" content="text/html; charset=utf-8">


</head>


<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">


<br class="">


<div>


<blockquote type="cite" class="">


<div class="">On 19 Jun 2017, at 10:43, Diana Picus via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a>> wrote:</div>


<br class="Apple-interchange-newline">


<div class="">On 17 June 2017 at 01:43, Quentin Colombet <<a href="mailto:qcolombet@apple.com" class="">qcolombet@apple.com</a>> wrote:<br class="">


<blockquote type="cite" class="">*** Your Help Is Needed ***<br class="">


<br class="">


- Please share your experience in using the GISel APIs and how we can make<br class="">


them better. Moving forward we’ll have those conversations on open source<br class="">


instead of internally/with a narrower audience.<br class="">


- Report any performance problem you identify<br class="">


- Propose patches!<br class="">


</blockquote>


<br class="">


While we're at it, these patches could use some review:<br class="">


* [GlobalISel] combine not symmetric merge/unmerge nodes [1]<br class="">


* [GlobalISel] Make multi-step legalization work [2] (<a href="https://reviews.llvm.org/D32529" class="">https://reviews.llvm.org/D32529</a>)</div>


</blockquote>


<div><br class="">


</div>


<div>As the author of the above patch, I think this one should be relatively easy to review.</div>


<br class="">


<blockquote type="cite" class="">


<div class="">* [GlobalISel] Enable specifying how to legalize non-power-of-2 size<br class="">


types. [NFC-ish] [3] (<a href="https://reviews.llvm.org/D30529" class="">https://reviews.llvm.org/D30529</a>)</div>


</blockquote>


<div><br class="">


</div>


<div>I'm working on rebasing this one on top of the above patch ("Make multi-step legalization work").</div>


<div>Most of that is done, but I'm still trying to write a better description of what this large patch actually</div>


<div>does, so that it becomes easier to review. Unfortunately, I haven't found a way to split that</div>


<div>patch up further in smaller incremental pieces.</div>


<div>I had put this on the back-burner and instead prioritized what was needed to enable GlobalISel</div>


<div>by default. Now that that is not being pushed as hard, I'll start looking again at it.</div>


<div>I hope to have an updated patch with a description that makes review easier towards the end of the week.</div>


<div><br class="">


</div>


<div>Thanks,</div>


<div><br class="">


</div>


<div>Kristof</div>


</div>


</body>


</html>