[PATCH] D133677: [llvm] Use x.empty() instead of llvm::empty(x) (NFC)
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 12 09:40:47 PDT 2022
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
hmm, I think I remember `llvm::empty` was added, maybe before llvm::iterator_range got an empty operation of its own? And in theory range-based-for loops only require begin/end, so it's sort of nice that empty could be used with that same requirement, but if, practically speaking, all our ranges have empty on them - yeah, guess this is an OK way forward.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133677/new/
https://reviews.llvm.org/D133677
More information about the llvm-commits
mailing list