[PATCH] D31582: Use caller information to avoid noalias metadata

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 2 10:48:05 PDT 2017


jdoerfert added a comment.

In https://reviews.llvm.org/D31582#716278, @dberlin wrote:

> It's early in the morning, but I'm having a little trouble understanding the overall goal.  Is this fixing a bug, it just being done because it's a waste of time to add the metadata in this situation.


The latter. There is no correctness bug, just to much metadata in certain situations. The initial idea was to improve the situation of the XLA folks that have to limit the amount of alias information they generate due to the compile time impact.
While David informed me this might not help their case at all, it should avoid the generation of metadata in some cases and could be extended to handle more.

At the end of the day this is just an idea I wanted to try out and see if folks like it and if it is worth pursuing.


https://reviews.llvm.org/D31582





More information about the llvm-commits mailing list