[llvm-profdata] Add check for text profile formats and improve error reporting
authorNathan Slingerland <slingn@gmail.com>
Thu, 12 Nov 2015 18:06:18 +0000 (18:06 +0000)
committerNathan Slingerland <slingn@gmail.com>
Thu, 12 Nov 2015 18:06:18 +0000 (18:06 +0000)
commit7b81b822ba8418a27225056bff9229b2d6c7dca9
tree2ec282665185e807215ed14b137fb80eb76a5703
parent7e02753319518a923db8d5460c860c6dd05d35ef
[llvm-profdata] Add check for text profile formats and improve error reporting

Summary:
This change addresses two possible instances of user error / confusion when
merging sampled profile data.

Previously any input that didn't match the raw or processed instrumented format
would automatically be interpreted as instrumented profile text format data.
No error would be reported during the merge.

Example:
If foo-sampled.profdata and bar-sampled.profdata are binary sampled profiles:

Old behavior:
$ llvm-profdata merge foo-sampled.profdata bar-sampled.profdata -output foobar-sampled.profdata
$ llvm-profdata show -sample foobar-sampled.profdata
error: foobar-sampled.profdata:1: Expected 'mangled_name:NUM:NUM', found  lprofi

This change adds basic checks for valid input data when assuming text input.
It also makes error messages related to file format validity more specific about
the assumbed profile data type.

New behavior:
$ llvm-profdata merge foo-sampled.profdata bar-sampled.profdata -o foobar-sampled.profdata
error: foo.profdata: Unrecognized instrumentation profile encoding format
Perhaps you forgot to use the -sample option?

Reviewers: bogner, davidxl, dnovillo

Subscribers: davidxl, llvm-commits

Differential Revision: http://reviews.llvm.org/D14558

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@252916 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/ProfileData/InstrProf.h
include/llvm/ProfileData/InstrProfReader.h
include/llvm/ProfileData/SampleProfReader.h
lib/ProfileData/InstrProf.cpp
lib/ProfileData/InstrProfReader.cpp
lib/ProfileData/SampleProf.cpp
lib/ProfileData/SampleProfReader.cpp
test/tools/llvm-profdata/raw-magic-but-no-header.test
test/tools/llvm-profdata/sample-profile-basic.test
test/tools/llvm-profdata/text-format-errors.test
tools/llvm-profdata/llvm-profdata.cpp