<div>I'm noticing that since I originally submitted this patch, a new keyword has been added... isExact. Particularly in llvm/include/llvm/IRBuilder.h, on the lshr and ashr operators.</div><div><br></div><div>I do not suppose that a circular shift can produce a poison value, but this is a little out of my domain. Would someone more familiar with poison values give me a bit of insight to them.</div>
<div><br></div><div>Ty,</div><div>Cameron<br><br><div class="gmail_quote">On Tue, Jul 31, 2012 at 8:05 AM, Cameron McInally <span dir="ltr"><<a href="mailto:cameron.mcinally@nyu.edu" target="_blank">cameron.mcinally@nyu.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oh, no. I should have been more clear. The patch was not rejected, just lost in the daily shuffle.<div><br></div><div>I already have my employer's approval to send this upstream, so I will prepare a patch against trunk this morning.<br>

<br><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>
> I proposed a similar patch to LLVM (left circular shift) around 10/2011.<br>
> Parts of my patch did make it into trunk about a year after, but others<br>
> did not.<br></div></blockquote><div><br></div></div><div>And now that I reread this... it should have been "month after", not "year after". </div></div></div>
</blockquote></div><br></div>