[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