<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="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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM.<br>
<div><div class="h5"><br>
On 2014 Mar 14, at 16:04, Eli Bendersky <<a href="mailto:eliben@google.com">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">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">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>