diff options
| author | Jordan Rose <jordan_rose@apple.com> | 2019-10-07 18:14:24 +0000 |
|---|---|---|
| committer | Jordan Rose <jordan_rose@apple.com> | 2019-10-07 18:14:24 +0000 |
| commit | fdaa74217420729140f1786ea037ac445a724c8e (patch) | |
| tree | 1b91bbb4f3cb6f58d9a1268c974d708fc7b1089b /llvm/lib/Transforms/IPO/PartialInlining.cpp | |
| parent | f385a381404797f5c44298d34c4ba804fad556d3 (diff) | |
| download | bcm5719-llvm-fdaa74217420729140f1786ea037ac445a724c8e.tar.gz bcm5719-llvm-fdaa74217420729140f1786ea037ac445a724c8e.zip | |
Second attempt to add iterator_range::empty()
Doing this makes MSVC complain that `empty(someRange)` could refer to
either C++17's std::empty or LLVM's llvm::empty, which previously we
avoided via SFINAE because std::empty is defined in terms of an empty
member rather than begin and end. So, switch callers over to the new
method as it is added.
https://reviews.llvm.org/D68439
llvm-svn: 373935
Diffstat (limited to 'llvm/lib/Transforms/IPO/PartialInlining.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/PartialInlining.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/IPO/PartialInlining.cpp b/llvm/lib/Transforms/IPO/PartialInlining.cpp index 62f4584d5f5..a0f0b6726cc 100644 --- a/llvm/lib/Transforms/IPO/PartialInlining.cpp +++ b/llvm/lib/Transforms/IPO/PartialInlining.cpp @@ -1264,7 +1264,7 @@ std::pair<bool, Function *> PartialInlinerImpl::unswitchFunction(Function *F) { if (PSI->isFunctionEntryCold(F)) return {false, nullptr}; - if (empty(F->users())) + if (F->users().empty()) return {false, nullptr}; OptimizationRemarkEmitter ORE(F); @@ -1370,7 +1370,7 @@ bool PartialInlinerImpl::tryPartialInline(FunctionCloner &Cloner) { return false; } - assert(empty(Cloner.OrigFunc->users()) && + assert(Cloner.OrigFunc->users().empty() && "F's users should all be replaced!"); std::vector<User *> Users(Cloner.ClonedFunc->user_begin(), |
