<div dir="ltr"><span style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif">Not sure what the protocol is for finding issues in patches that have already been committed - should I be posting to parallel_libs-dev instead?</span><br><div><span style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif"><br></span></div><div><font color="#212121" face="helvetica neue, helvetica, arial, sans-serif">I think either place is a good choice for this kind of issue. Thanks for finding this. I'll get a fix in today and add some unit tests to make sure it is working :).</font></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 24, 2016 at 11:56 AM James Price <<a href="mailto:j.price@bristol.ac.uk">j.price@bristol.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jprice added a comment.<br>
<br>
Not sure what the protocol is for finding issues in patches that have already been committed - should I be posting to parallel_libs-dev instead?<br>
<br>
This issue only showed up when I tried to actually write some code that uses all this stuff.<br>
<br>
<br>
================<br>
Comment at: parallel-libs/trunk/streamexecutor/include/streamexecutor/Executor.h:44<br>
@@ +43,3 @@<br>
+  Expected<GlobalDeviceMemory<T>> allocateDeviceMemory(size_t ElementCount) {<br>
+    return PExecutor->allocateDeviceMemory(ElementCount * sizeof(T));<br>
+  }<br>
----------------<br>
There's a mismatch between the return types of these two `allocateDeviceMemory` methods. The `PlatformExecutor` version returns a `GlobalDeviceMemoryBase`, whereas this one tries to return a `GlobalDeviceMemory<T>`.<br>
<br>
This causes compilation failure if this `Executor::allocateDeviceMemory<T>()` method is actually used. Because this method is templated, it needs to actually be called somewhere for the issue to show up, and it doesn't appear to be covered in the unit tests.<br>
<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D23577" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23577</a><br>
<br>
<br>
<br>
</blockquote></div>