[PATCH] D15852: [PGO]: Simplify coverage data lowering code

[Vedant Kumar via llvm-commits]

vsk added a comment.

I have a question regarding this:

  Besides, in the near future, when name compression is implemented, the name references won't be available from the coverage data anymore.

Is the plan to change the FunctionRecord format by replacing `Name` with an offset into the `CoverageNamesVar`? If so, the temporary IR bloat of having a names array as well as complete function records would be OK.


http://reviews.llvm.org/D15852