PATCH: Consistent use of the noduplicate attribute

James Molloy james at jamesmolloy.co.uk
Mon Mar 17 03:21:09 PDT 2014


Hi Eli,

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.

I'd appreciate if that could be added back in!

Cheers,

James


On 15 March 2014 22:12, Duncan P. N. Exon Smith <dexonsmith at apple.com>wrote:

> LGTM.
>
> On 2014 Mar 14, at 16:04, Eli Bendersky <eliben at google.com> wrote:
>
> > Hello,
> >
> > 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.
> >
> > In addition, setCannotDuplicate() is removed because it's not being used
> anywhere in the codebase.
> >
> > PTAL,
> > Eli
> >
> >
> <noduplicate-consistent-use.1.patch>_______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140317/da4f90c3/attachment.html>


More information about the llvm-commits mailing list