[llvm-profdata] Add check for text profile formats and improve error reporting (2nd...
authorNathan Slingerland <slingn@gmail.com>
Fri, 13 Nov 2015 03:47:58 +0000 (03:47 +0000)
committerNathan Slingerland <slingn@gmail.com>
Fri, 13 Nov 2015 03:47:58 +0000 (03:47 +0000)
commit572e633224a4b84ac30176fc94317a9f63ae238e
tree77ed9ab41835c1386d2c232a390da6f3d14bf219
parent1adc6064bc8555c1fd505f91e509bc4e6e7e6aa2
[llvm-profdata] Add check for text profile formats and improve error reporting (2nd try)

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@253009 91177308-0d34-0410-b5e6-96231b3b80d8
12 files changed:
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/Transforms/SampleProfile/syntax.ll
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