[PATCH] Change loop unroll and vectorizer metadata prefix to 'llvm.loop.'

Pekka Jääskeläinen pekka.jaaskelainen at tut.fi
Tue Jun 24 03:31:56 PDT 2014


Hi,

LGTM. One thing though: If one of the renamed metadatas was there
already in LLVM 3.4, I think it's polite for
out-of-tree projects (that might refer to the metadata by name) to
have this rename mentioned in the 3.5 release notes.

On Tue, Jun 24, 2014 at 12:30 AM, Mark Heffernan <meheff at google.com> wrote:
> These patches rename the loop unrolling and loop vectorizer metadata
> such that they have a common 'llvm.loop.' prefix.  Metadata name
> changes:
>
> llvm.vectorizer.* => llvm.loop.vectorizer.*
> llvm.loopunroll.* => llvm.loop.unroll.*
>
> This was a suggestion from an earlier review
> (http://reviews.llvm.org/D4090) which added the loop unrolling
> metadata.  Two patches are attached.  One for front-end and one for
> optimizer.
>
> Mark



-- 
--PJ



More information about the cfe-commits mailing list