[llvm] r237341 - [TableGen] Remove ListInit::size() in favor of getSize() which does the same thing and is already used in most places. NFC.
Craig Topper
craig.topper at gmail.com
Thu May 14 14:40:53 PDT 2015
Agreed, but it touched a lot less places. Though a lot of uses of getSize
should be range-based for loops. Maybe I'll just go fix those.
On Thu, May 14, 2015 at 1:50 PM, Owen Anderson <resistor at mac.com> wrote:
> Why standardize on this one? size() is more consistent with other LLVM
> data types.
>
> —Owen
>
> > On May 13, 2015, at 10:53 PM, Craig Topper <craig.topper at gmail.com>
> wrote:
> >
> > Author: ctopper
> > Date: Thu May 14 00:53:56 2015
> > New Revision: 237341
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=237341&view=rev
> > Log:
> > [TableGen] Remove ListInit::size() in favor of getSize() which does the
> same thing and is already used in most places. NFC.
> >
> > Modified:
> > llvm/trunk/include/llvm/TableGen/Record.h
> > llvm/trunk/utils/TableGen/CodeGenRegisters.cpp
> >
> > Modified: llvm/trunk/include/llvm/TableGen/Record.h
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/TableGen/Record.h?rev=237341&r1=237340&r2=237341&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/include/llvm/TableGen/Record.h (original)
> > +++ llvm/trunk/include/llvm/TableGen/Record.h Thu May 14 00:53:56 2015
> > @@ -842,7 +842,6 @@ public:
> > inline const_iterator begin() const { return Values.begin(); }
> > inline const_iterator end () const { return Values.end(); }
> >
> > - inline size_t size () const { return Values.size(); }
> > inline bool empty() const { return Values.empty(); }
> >
> > /// resolveListElementReference - This method is used to implement
> >
> > Modified: llvm/trunk/utils/TableGen/CodeGenRegisters.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenRegisters.cpp?rev=237341&r1=237340&r2=237341&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/utils/TableGen/CodeGenRegisters.cpp (original)
> > +++ llvm/trunk/utils/TableGen/CodeGenRegisters.cpp Thu May 14 00:53:56
> 2015
> > @@ -676,7 +676,7 @@ CodeGenRegisterClass::CodeGenRegisterCla
> > // Allocation order 0 is the full set. AltOrders provides others.
> > const SetTheory::RecVec *Elements = RegBank.getSets().expand(R);
> > ListInit *AltOrders = R->getValueAsListInit("AltOrders");
> > - Orders.resize(1 + AltOrders->size());
> > + Orders.resize(1 + AltOrders->getSize());
> >
> > // Default allocation order always contains all registers.
> > for (unsigned i = 0, e = Elements->size(); i != e; ++i) {
> > @@ -689,7 +689,7 @@ CodeGenRegisterClass::CodeGenRegisterCla
> >
> > // Alternative allocation orders may be subsets.
> > SetTheory::RecSet Order;
> > - for (unsigned i = 0, e = AltOrders->size(); i != e; ++i) {
> > + for (unsigned i = 0, e = AltOrders->getSize(); i != e; ++i) {
> > RegBank.getSets().evaluate(AltOrders->getElement(i), Order,
> R->getLoc());
> > Orders[1 + i].append(Order.begin(), Order.end());
> > // Verify that all altorder members are regclass members.
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
--
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150514/86fb4823/attachment.html>
More information about the llvm-commits
mailing list