[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