[PATCH] D47267: [UnrollAndJam] Add unroll_and_jam pragma handling

[Michael Kruse via Phabricator via cfe-commits]

Meinersbur added a comment.

In https://reviews.llvm.org/D47267#1122425, @dmgreen wrote:

> I noticed in the paper that you used the name "unrollandjam", minus underscores. Should I change this use that spelling here? I have no strong opinion of one over the other (was just using what I had found from the Intel docs).


IMHO you can keep `unroll_and_jam` (which is already supported by Intel syntax). When I imagined the name, I had xlc's `unrollandfuse` in mind, but found that "and jam" is better known than "and fuse".

We can have a discussion about how to name them in general. `nounroll_and_jam` seems a strange mix of write words together and separate words by underscores.


https://reviews.llvm.org/D47267