<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 27, 2015 at 9:08 AM, David Chisnall <span dir="ltr"><<a href="mailto:David.Chisnall@cl.cam.ac.uk" target="_blank">David.Chisnall@cl.cam.ac.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 27 Mar 2015, at 15:47, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
> Here's one example from the gep instruction changes (none of the test cases in that commit were updated manually, all were done using the script)<br>
><br>
> -  %arrayidx = getelementptr <2 x i32> addrspace(1)* %tmp, i32 %tmp1<br>
> +  %arrayidx = getelementptr <2 x i32>, <2 x i32> addrspace(1)* %tmp, i32 %tmp1<br>
><br>
> Is there some other use of addrspace you're referring to/I'm not understanding?<br>
<br>
</span>That looks like the form that we were seeing, but it ended up being translated with the addrspace(1) bit before the comma as well.<br></blockquote><div><br>My best guess is that I might not've accounted for all possible forms of whitespace - if you have an example I can try to improve the script & send it as a follow-up to the original commit message. Sorry for the inconvenience. :/<br><br>- David<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
David<br>
<br>
</font></span></blockquote></div><br></div></div>