[PATCH] D31900: [AddDiscriminators] Assign discriminators to memset/memcpy/memmove intrinsic calls.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 10 19:19:33 PDT 2017


danielcdh added a comment.

I don't quite understand why you want to set a new discriminator for memcpy builtin. What optimization would it enable? For normal function calls, we need to have discriminator to distinguish callsites in the same BB so that we can annotate the inlined callee correctly. But for the memcpy case, looks like adding a new discriminator does not help down-stream optimizations like inlining?


https://reviews.llvm.org/D31900





More information about the llvm-commits mailing list