[CSSPGO][llvm-profgen] Filter out the instructions without location info for symbolizer
authorwlei <wlei@fb.com>
Wed, 10 Feb 2021 18:04:39 +0000 (10:04 -0800)
committerwlei <wlei@fb.com>
Sat, 13 Feb 2021 00:47:49 +0000 (16:47 -0800)
commitafd8bd601eaf268357e2cc0cf4c0382732fcf923
tree8f083e240c18dc41147a107b89423023a131a520
parent964f8103c58db5917c66ef39f50acf2004c90144
[CSSPGO][llvm-profgen] Filter out the instructions without location info for symbolizer

It appears some instructions doesn't have the debug location info and the symbolizer will return an empty call stack for them which will cause some crash later in profile unwinding. Actually we do not record the sample info for them, so this change just filter out those instruction.

As those instruction would appears at the begin and end of the instruction list, without them we need to add the boundary check for IP `advance` and `backward`.

Also for pseudo probe based profile, we actually don't need the symbolized location info, so here just change to use an empty stack for it. This could save half of the binary loading time.

Differential Revision: https://reviews.llvm.org/D96434
llvm/test/tools/llvm-profgen/inline-cs-noprobe.test
llvm/test/tools/llvm-profgen/noinline-cs-noprobe.test
llvm/test/tools/llvm-profgen/recursion-compression-noprobe.test
llvm/tools/llvm-profgen/PerfReader.cpp
llvm/tools/llvm-profgen/ProfileGenerator.cpp
llvm/tools/llvm-profgen/ProfiledBinary.cpp
llvm/tools/llvm-profgen/ProfiledBinary.h