[PATCH] D18555: [ThinLTO] Add option to dump value name to GUID mapping

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 1 12:38:33 PDT 2016


tejohnson added a comment.

In http://reviews.llvm.org/D18555#389713, @joker.eph wrote:

> That's close to what I implemented locally when I was debugging this code last month.
>  My plan to have a "nice" solution was to dump a map in a file that could be loaded by other pieces. For instance you could use the map file by doing `opt -function-import ....    -debug -debug-map-guid=path/to/map`  and expose an API that convert GUID to name.




In http://reviews.llvm.org/D18555#389715, @joker.eph wrote:

> Maybe on the short term it is not worth investing time in this and I won't oppose to add the easy solution you have.


Yea, I thought about something along those lines too, but decided the simple solution was sufficient for now. Note that this patch would be a subset of that approach anyway since it gives you the map that would be fed back in.


http://reviews.llvm.org/D18555





More information about the llvm-commits mailing list