Skip to content

Commit

Permalink
[NFC] Fix the signature and definition of findByPrefix
Browse files Browse the repository at this point in the history
In https://reviews.llvm.org/rG257b29715bb27b7d9f6c3c40c481b6a4af0b37e5,
the definition of OptTable::Info::Flags was changed from `unsigned
short` to `unsigned int`, but the definition/declaration of
OptTable::findByPrefix wasn't updated to reflect that.

This patch updates findByPrefix accordingly.
  • Loading branch information
banach-space committed Sep 11, 2020
1 parent 26d9a94 commit 4eed800
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion llvm/include/llvm/Option/OptTable.h
Expand Up @@ -152,7 +152,7 @@ class OptTable {
///
/// \return The vector of flags which start with Cur.
std::vector<std::string> findByPrefix(StringRef Cur,
unsigned short DisableFlags) const;
unsigned int DisableFlags) const;

/// Find the OptTable option that most closely matches the given string.
///
Expand Down
2 changes: 1 addition & 1 deletion llvm/lib/Option/OptTable.cpp
Expand Up @@ -228,7 +228,7 @@ OptTable::suggestValueCompletions(StringRef Option, StringRef Arg) const {
}

std::vector<std::string>
OptTable::findByPrefix(StringRef Cur, unsigned short DisableFlags) const {
OptTable::findByPrefix(StringRef Cur, unsigned int DisableFlags) const {
std::vector<std::string> Ret;
for (size_t I = FirstSearchableIndex, E = OptionInfos.size(); I < E; I++) {
const Info &In = OptionInfos[I];
Expand Down

0 comments on commit 4eed800

Please sign in to comment.