diff options
| author | Kazu Hirata <kazu@google.com> | 2021-01-18 10:16:34 -0800 |
|---|---|---|
| committer | Kazu Hirata <kazu@google.com> | 2021-01-18 10:16:34 -0800 |
| commit | dc300beba7a849aac44c39ccc450a575db99bc14 (patch) | |
| tree | 2859643b216779a1a502ad7a02c5b2cffd07be65 /llvm/lib/Transforms/Utils/LoopUtils.cpp | |
| parent | 28ea50f524b56e11b608ca1f768d2981579ebe75 (diff) | |
| download | llvm-dc300beba7a849aac44c39ccc450a575db99bc14.zip llvm-dc300beba7a849aac44c39ccc450a575db99bc14.tar.gz llvm-dc300beba7a849aac44c39ccc450a575db99bc14.tar.bz2 | |
[STLExtras] Add a default value to drop_begin
This patch adds the default value of 1 to drop_begin.
In the llvm codebase, 70% of calls to drop_begin have 1 as the second
argument. The interface similar to with std::next should improve
readability.
This patch converts a couple of calls to drop_begin as examples.
Differential Revision: https://reviews.llvm.org/D94858
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUtils.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/LoopUtils.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/LoopUtils.cpp b/llvm/lib/Transforms/Utils/LoopUtils.cpp index 401fe45..e6575ee 100644 --- a/llvm/lib/Transforms/Utils/LoopUtils.cpp +++ b/llvm/lib/Transforms/Utils/LoopUtils.cpp @@ -349,7 +349,7 @@ Optional<MDNode *> llvm::makeFollowupLoopID( bool Changed = false; if (InheritAllAttrs || InheritSomeAttrs) { - for (const MDOperand &Existing : drop_begin(OrigLoopID->operands(), 1)) { + for (const MDOperand &Existing : drop_begin(OrigLoopID->operands())) { MDNode *Op = cast<MDNode>(Existing.get()); auto InheritThisAttribute = [InheritSomeAttrs, @@ -386,7 +386,7 @@ Optional<MDNode *> llvm::makeFollowupLoopID( continue; HasAnyFollowup = true; - for (const MDOperand &Option : drop_begin(FollowupNode->operands(), 1)) { + for (const MDOperand &Option : drop_begin(FollowupNode->operands())) { MDs.push_back(Option.get()); Changed = true; } |