Speculative fix for build failures due to consumeInteger.
authorZachary Turner <zturner@google.com>
Thu, 22 Sep 2016 15:55:05 +0000 (15:55 +0000)
committerZachary Turner <zturner@google.com>
Thu, 22 Sep 2016 15:55:05 +0000 (15:55 +0000)
commitd5d57635baf6f3aa8d937e20f43dad75179f2cee
tree49fd94438cbfb61edf2522d5f67bc1f098a18fce
parent74c3fd17984737a0ef87fd3a83eb035fc58adf21
Speculative fix for build failures due to consumeInteger.

A recent patch added support for consumeInteger() and made
getAsInteger delegate to this function.  A few buildbots are
failing as a result with an assertion failure.  On a hunch,
I tested what happens if I call getAsInteger() on an empty
string, and sure enough it crashes the same way that the
buildbots are crashing.

I confirmed that getAsInteger() on an empty string did not
crash before my patch, so I suspect this to be the cause.

I also added a unit test for the empty string.

llvm-svn: 282170
llvm/lib/Support/StringRef.cpp
llvm/unittests/ADT/StringRefTest.cpp