search for: immutablelist

On Jul 25, 2011, at 10:58 PM, Talin wrote: > To handle the fact that types do not (and can not, at least as long as we intend to support obscure languages like "C" :) have linkage, the the linker uses a "best effort" approach. It attempts to merge types and rewrite IR to use the merged types where it can, but it doesn't make any guarantees. > > I want to add an

On Jul 4, 2011, at 7:53 PM, Nick Lewycky wrote: > Rui Paulo wrote: >> Hi, >> >> Attached is a patch to add a contains method to ImmutableList. I need this in the clang Static Analyzer. > > Do you really need "this->" here?: > > + for (iterator I = this->begin(), E = this->end(); I != E; ++I) { Oops, I guess not. > > Yes the type is templated, but it doesn't have a template base. > >...

I just filed: Add contains() to ImmutableList - http://llvm.org/bugs/show_bug.cgi?id=10271 Is the patch acceptable? Regards, -- Rui Paulo

...would fit on one line but they weren't put on one line); this is more than the total number of one-line definitions. To obtain a lower bound on the cited "hundreds", I clang-format'd everything and then looked for just diff chunks similar to: -template <typename T> -class ImmutableList { +template <typename T> class ImmutableList { $ cd llvm/ $ clang-format -i **/*.cpp **/*.h $ git diff | grep -B2 '^+template' | egrep -B1 '^-(struct|class)' | grep '^-template' | wc -l 287 # For comparison $ git grep '^\s*template' -- '*.cpp' '*.h...

...lang-format chose one or the other as the default) then, sure, flip a coin & go with it. > To obtain a lower bound on the cited > "hundreds", I clang-format'd everything and then looked for just diff chunks > similar to: > > -template <typename T> > -class ImmutableList { > +template <typename T> class ImmutableList { > > $ cd llvm/ > $ clang-format -i **/*.cpp **/*.h > $ git diff | grep -B2 '^+template' | egrep -B1 '^-(struct|class)' | grep > '^-template' | wc -l > 287 > # For comparison > $ git grep '^...

On Mon, Jul 1, 2013 at 2:38 PM, Sean Silva <silvas at purdue.edu> wrote: > > > > On Mon, Jul 1, 2013 at 2:31 PM, David Blaikie <dblaikie at gmail.com> wrote: >> >> >> Have you got any statistics for the current state of LLVM with respect >> to this formatting issue? If something is already the overwhelmingly >> common style (& it's not

On Mon, Jul 1, 2013 at 2:31 PM, David Blaikie <dblaikie at gmail.com> wrote: > > > Have you got any statistics for the current state of LLVM with respect > to this formatting issue? If something is already the overwhelmingly > common style (& it's not a case where it used to be the style, the > style has been updated, and nothing has been migrated yet) then just

On 29 July 2011 12:48, Dan Bailey <dan at dneg.com> wrote: > I'm not going to comment on the syntax of any of the proposals made, just > dispute that they don't provide sufficient value to make them worth > introducing. Hi Dan, I've read the arguments on both sides and I have to say I'm with Chris on this one. I normally keep GDB running with my debug image. After

...2.8/include/llvm/ADT/EquivalenceClasses.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/FoldingSet.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/GraphTraits.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/ImmutableIntervalMap.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/ImmutableList.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/ImmutableMap.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/ImmutableSet.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/IndexedMap.h -- Installing: /usr/local/llvm-2.8/include/llvm/ADT/IntrusiveRefCntPtr.h -- Installing: /usr/local...

On Thu, Sep 30, 2010 at 3:08 PM, Samuel Williams <space.ship.traveller at gmail.com> wrote: > Hi, > > I might just be doing something stupid, but when I do > > $ cmake -DCMAKE_INSTALL_PREFIX=/usr/local/llvm-2.8 -DCMAKE_BUILD_TYPE=Release .. > $ sudo make install > > I don't get the expected headers in >        /usr/local/llvm-2.8/include/llvm > > It is

Hi, I might just be doing something stupid, but when I do $ cmake -DCMAKE_INSTALL_PREFIX=/usr/local/llvm-2.8 -DCMAKE_BUILD_TYPE=Release .. $ sudo make install I don't get the expected headers in /usr/local/llvm-2.8/include/llvm It is simply an empty directory. What am I doing wrong? This is on Mac OS X, CMake 2.8+ Kind regards, Samuel