<div dir="ltr">Thanks for the review - committed in r204049.<div><br></div><div>James - I revived the setter per your request (and put it in the invoke too, for consistency with the call).</div><div><br></div><div>Eli<br><div class="gmail_extra">

<br><br><div class="gmail_quote">On Mon, Mar 17, 2014 at 3:21 AM, James Molloy <span dir="ltr"><<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr">Hi Eli,<div><br></div><div>Sorry I didn't get to this very quickly. The patch in general looks fine but I don't agree with removing setCannotDuplicate(). Many users of noduplicate (for example OpenCL stacks) are out of tree, and I don't think it's right to make an interface unorthogonal (getter but no setter) when it's so simple just because there are no users in tree.</div>


<div><br></div><div>I'd appreciate if that could be added back in!</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><div class=""><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">

On 15 March 2014 22:12, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">LGTM.<br>
<div><div><br>
On 2014 Mar 14, at 16:04, Eli Bendersky <<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>> wrote:<br>
<br>
> Hello,<br>
><br>
> The "noduplicate" attribute of call instructions is sometimes queried directly and sometimes through the cannotDuplicate() predicate. This patch streamlines all queries to use the cannotDuplicate() predicate. It also adds this predicate to InvokeInst, to mirror what CallInst has.<br>



><br>
> In addition, setCannotDuplicate() is removed because it's not being used anywhere in the codebase.<br>
><br>
> PTAL,<br>
> Eli<br>
><br>
</div></div>> <noduplicate-consistent-use.1.patch>_______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div>