dmgreen added a comment.

I quite like the UnrollAndFuse naming. I'd not heard that the xlc compiler 
called it that. The UnrollAndJam pass was origin named that before I renamed 
for similar reasons (UnrollAndJam being more well known).

I see your point about the mix of underscores. "nounroll_and_jam" also comes 
from the Intel docs, and theres already "nounroll" vs "unroll". The "no" 
becomes a qualifier on "unroll_and_jam". "no_unroll_and_jam" feels less 
consistent to me.

But if you have a strong opinion, I'm happy enough to change it.


https://reviews.llvm.org/D47267



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to