diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-01-15 11:41:30 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-01-15 11:41:30 +0000 |
| commit | 8c0809c7f8a6b8ae8d6b953f9a30a56ec584ff56 (patch) | |
| tree | 98e41eb51466fc2b615e8acf9011cc4076cdd05e /llvm/lib/CodeGen/SelectionDAG | |
| parent | 8ca43224db45c489cf7bf765d057f3d1a92be92b (diff) | |
| download | bcm5719-llvm-8c0809c7f8a6b8ae8d6b953f9a30a56ec584ff56.tar.gz bcm5719-llvm-8c0809c7f8a6b8ae8d6b953f9a30a56ec584ff56.zip | |
Replace size method call of containers to empty method where appropriate
This patch was generated by a clang tidy checker that is being open sourced.
The documentation of that checker is the following:
/// The emptiness of a container should be checked using the empty method
/// instead of the size method. It is not guaranteed that size is a
/// constant-time function, and it is generally more efficient and also shows
/// clearer intent to use empty. Furthermore some containers may implement the
/// empty method but not implement the size method. Using empty whenever
/// possible makes it easier to switch to another container in the future.
Patch by Gábor Horváth!
llvm-svn: 226161
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp b/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp index 72e0aca8408..a1f425eeac1 100644 --- a/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp @@ -2340,7 +2340,7 @@ TargetLowering::AsmOperandInfoVector TargetLowering::ParseConstraints( } // If we have multiple alternative constraints, select the best alternative. - if (ConstraintOperands.size()) { + if (!ConstraintOperands.empty()) { if (maCount) { unsigned bestMAIndex = 0; int bestWeight = -1; |

