Skip to content

Commit

Permalink
[LifetimeAnalysis] Add [[gsl::Pointer]] to llvm::StringRef
Browse files Browse the repository at this point in the history
Summary:
This detected the bugs fixed in
  https://reviews.llvm.org/D66442
and
  https://reviews.llvm.org/D66440

The warning itself was implemented in
  https://reviews.llvm.org/D63954
  https://reviews.llvm.org/D64256
  https://reviews.llvm.org/D65120
  https://reviews.llvm.org/D65127
  https://reviews.llvm.org/D66152

Reviewers: zturner, mehdi_amini, gribozavr

Subscribers: dexonsmith, Szelethus, xazax.hun, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D66443
  • Loading branch information
mgehre committed Apr 16, 2020
1 parent 0741dec commit 83d5131
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/include/llvm/ADT/StringRef.h
Expand Up @@ -54,7 +54,7 @@ namespace llvm {
/// situations where the character data resides in some other buffer, whose
/// lifetime extends past that of the StringRef. For this reason, it is not in
/// general safe to store a StringRef.
class StringRef {
class [[gsl::Pointer]] StringRef {
public:
static const size_t npos = ~size_t(0);

Expand Down

0 comments on commit 83d5131

Please sign in to comment.