[PATCH] D23549: [SROA] Preserve llvm.mem.parallel_loop_access metadata

Dorit Nuzman via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 28 03:57:20 PDT 2016


dorit added a comment.

In https://reviews.llvm.org/D23549#527278, @chandlerc wrote:

>


...

> I can absolutely believe we're dropping metadata when we rewrite stuff but fail to promote it to registers. I think the rewriting logic here never got the most careful of analysis because we "usually" think of it as rewriting in preparation for promotion, but we definitely have lots of rewrites at this point that don't involve in promotion so we should probably systematically go through this.


Agreed... Is it ok though to commit this patch, as a tiny step towards that (which allows the mem.parallel tags reach the vectorizer) ?


https://reviews.llvm.org/D23549





More information about the llvm-commits mailing list