In a discussion on IRC, Micheal was complaining about the semantics of StringSet, specifically how the iterators work. DannyB mentioned that we have a Trie class, so I decided to check it out. Lo and behold, I couldn't find _any_ uses of it, and when I deleted it from my system and did a clean rebuild, everything was fine. Looking at the log, the last substantial change to this class was in 2007. So - is anyone using this file/class? Is anyone planning on using it in the future? Should I just delete it? -- Marshall Marshall Clow Idio Software <mailto:mclow.lists at gmail.com> A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait). -- Yu Suzuki
Delete it. =] Thanks. On Tue, Sep 4, 2012 at 9:53 PM, Marshall Clow <mclow.lists at gmail.com> wrote:> In a discussion on IRC, Micheal was complaining about the semantics of > StringSet, specifically how the iterators work. > > DannyB mentioned that we have a Trie class, so I decided to check it out. > Lo and behold, I couldn't find _any_ uses of it, and when I deleted it > from my system and did a clean rebuild, everything was fine. > > Looking at the log, the last substantial change to this class was in 2007. > > So - is anyone using this file/class? > Is anyone planning on using it in the future? > Should I just delete it? > > -- Marshall > > Marshall Clow Idio Software <mailto:mclow.lists at gmail.com> > > A.D. 1517: Martin Luther nails his 95 Theses to the church door and is > promptly moderated down to (-1, Flamebait). > -- Yu Suzuki > > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120904/900ff0ec/attachment.html>
> So - is anyone using this file/class?No. It was used one day in tablegen. -- With best regards, Anton Korobeynikov Faculty of Mathematics and Mechanics, Saint Petersburg State University
Yes, please zap it. -Chris On Sep 4, 2012, at 7:05 PM, Chandler Carruth <chandlerc at google.com> wrote:> Delete it. =] Thanks. > > > On Tue, Sep 4, 2012 at 9:53 PM, Marshall Clow <mclow.lists at gmail.com> wrote: > In a discussion on IRC, Micheal was complaining about the semantics of StringSet, specifically how the iterators work. > > DannyB mentioned that we have a Trie class, so I decided to check it out. > Lo and behold, I couldn't find _any_ uses of it, and when I deleted it from my system and did a clean rebuild, everything was fine. > > Looking at the log, the last substantial change to this class was in 2007. > > So - is anyone using this file/class? > Is anyone planning on using it in the future? > Should I just delete it? > > -- Marshall > > Marshall Clow Idio Software <mailto:mclow.lists at gmail.com> > > A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait). > -- Yu Suzuki > > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120905/0d0e5d9d/attachment.html>
Apparently Analagous Threads
- [LLVMdev] /llvm/include/ADT/Trie.h?
- [LLVMdev] Binutils and LLVM - gathering information
- [LLVMdev] NIT: Include guards for include/llvm/Support/COFF.h
- [LLVMdev] Questions about llvm/Object/COFF.h
- [LLVMdev] Anybody translating the LLVM FAQ from HTML to Sphinx?