[PATCH] D28965: [PGO] Value profile for size of memory intrinsic calls

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 10:36:00 PST 2017


davidxl added a comment.

See my comment in https://reviews.llvm.org/D28964.  There is no need to introduce runtime change nor new clang intrinsics.   The instrumentation lower can do the job of mapping the ranges. Another choice is to let the instrumenter does the range mapping work and keep the lowerer unchanged.


https://reviews.llvm.org/D28965





More information about the llvm-commits mailing list