Rename metadata llvm.loop.vectorize.unroll to llvm.loop.vectorize.interleave

[Mark Heffernan]

Here's the updated patches.  llvm.loop.vectorize.interleave is now
llvm.loop.interleave.count.

Mark

On Fri, Jul 18, 2014 at 2:25 PM, Mark Heffernan <meheff at google.com> wrote:

> On Fri, Jul 18, 2014 at 2:11 PM, Tyler Nowicki <tnowicki at apple.com> wrote:
>
>> Perhaps you could rename 'loop.vectorize.unroll' to
>> 'loop.interleave.count'. That way in a future patch we could add
>> 'loop.interleave.enable'.
>>
>> What do you think?
>>
>
> Sounds reasonable.  Happy to change it to whatever is most appropriate.
>
> Mark
>
>
>>
>> Tyler
>>
>> On Jul 18, 2014, at 1:51 PM, Hal Finkel <hfinkel at anl.gov> wrote:
>>
>> > Nadav, Arnold,
>> >
>> > I think this makes sense. Do either of you object?
>> >
>> > -Hal
>> >
>> > ----- Original Message -----
>> >> From: "Mark Heffernan" <meheff at google.com>
>> >> To: llvm-commits at cs.uiuc.edu, "cfe-commits" <cfe-commits at cs.uiuc.edu>,
>> "Hal Finkel" <hfinkel at anl.gov>, "Tyler
>> >> Nowicki" <tnowicki at apple.com>
>> >> Sent: Friday, July 18, 2014 3:44:00 PM
>> >> Subject: Rename metadata llvm.loop.vectorize.unroll to
>> llvm.loop.vectorize.interleave
>> >>
>> >>
>> >> This patch rename metadata llvm.loop.vectorize.unroll to
>> >> llvm.loop.vectorize.interleave to avoid confusion with the
>> >> concatenation unroller metadata (llvm.loop.unroll). It fell out of a
>> >> discussion on patch http://reviews.llvm.org/D4576 . On that patch
>> >> there was some confusion about whether llvm.loop.vectorize.unroll
>> >> controlled the traditional (concatenation) loop unroller. And IIRC
>> >> this isn't the first time there has been this confusion. Renaming
>> >> vectorize.unroll to vectorize.interleave also more closely matches
>> >> the pragma which generates the metadata (#pragma clang loop
>> >> interleave_count). There are still lots of uses of "unroll" within
>> >> the code in function/variable names and comments. I have no
>> >> intention of changing those.
>> >>
>> >>
>> >> Mark
>> >
>> > --
>> > Hal Finkel
>> > Assistant Computational Scientist
>> > Leadership Computing Facility
>> > Argonne National Laboratory
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140718/549e8293/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename.llvm.patch
Type: text/x-patch
Size: 4010 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140718/549e8293/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename.clang.patch
Type: text/x-patch
Size: 4010 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140718/549e8293/attachment-0001.bin>