search for: keyinfot

...on */ }; } with the following function definition: void DataFlowValue::set(DMTy emap) { ExprMap = *emap; //Line153: } When I compile, I get the following error: /home/zeus/masterLLVM/llvm/include/llvm/ADT/DenseMap.h: In member function ‘void llvm::DenseMapBase<DerivedT, KeyT, ValueT, KeyInfoT>::destroyAll() [with DerivedT = llvm::DenseMap<{anonymous}::Expression, unsigned int>, KeyT = {anonymous}::Expression, ValueT = unsigned int, KeyInfoT = llvm::DenseMapInfo<{anonymous}::Expression>]’: /home/zeus/masterLLVM/llvm/include/llvm/ADT/DenseMap.h:600:5: instantiated from ‘voi...

...definition: > > void DataFlowValue::set(DMTy emap) { > ExprMap = *emap; //Line153: > } > > When I compile, I get the following error: > > /home/zeus/masterLLVM/llvm/include/llvm/ADT/DenseMap.h: In member function > ‘void llvm::DenseMapBase<DerivedT, KeyT, ValueT, KeyInfoT>::destroyAll() > [with DerivedT = llvm::DenseMap<{anonymous}::Expression, unsigned int>, KeyT > = {anonymous}::Expression, ValueT = unsigned int, KeyInfoT = > llvm::DenseMapInfo<{anonymous}::Expression>]’: > /home/zeus/masterLLVM/llvm/include/llvm/ADT/DenseMap.h:600:5:...

...nd it'll write out to the file, "Hello Wor" and then seg fault without assert. Sadly, this only happens for certain C code, any idea why this might be occuring? Also, during build I get a lot of these warnings:llvm/ADT/DenseMap.h: In copy constructor ‘llvm::DenseMap<KeyT, ValueT, KeyInfoT>::DenseMap(const llvm::DenseMap<KeyT, ValueT, KeyInfoT>&) [with KeyT = unsigned int, ValueT = llvm::PointerAlignElem, KeyInfoT = llvm::DenseMapInfo<unsigned int>]’: I'm guessing I messed up when updating somehow? I just used gmake update. Thanks. -------------- next part --...

...e, "Hello Wor" and then seg fault without > assert. > > Sadly, this only happens for certain C code, any idea why this might be > occuring? > > Also, during build I get a lot of these warnings:llvm/ADT/DenseMap.h: In > copy constructor ‘llvm::DenseMap<KeyT, ValueT, KeyInfoT>::DenseMap(const > llvm::DenseMap<KeyT, ValueT, KeyInfoT>&) [with KeyT = unsigned int, ValueT > = llvm::PointerAlignElem, KeyInfoT = llvm::DenseMapInfo<unsigned int>]’: > > I'm guessing I messed up when updating somehow? I just used gmake update. > > Thanks....

...then seg fault without >> assert. >> >> Sadly, this only happens for certain C code, any idea why this might be >> occuring? >> >> Also, during build I get a lot of these warnings:llvm/ADT/DenseMap.h: In >> copy constructor ‘llvm::DenseMap<KeyT, ValueT, KeyInfoT>::DenseMap(const >> llvm::DenseMap<KeyT, ValueT, KeyInfoT>&) [with KeyT = unsigned int, ValueT >> = llvm::PointerAlignElem, KeyInfoT = llvm::DenseMapInfo<unsigned int>]’: >> >> I'm guessing I messed up when updating somehow? I just used gmake update. &...

Hi, After building out project in release mode, caught an assertion, which we have not seen before: hello_f: /tmp/rpmbuild_debug/BUILD/llvm/build/include/llvm/ADT/DenseMap.h:126: void llvm::DenseMap<KeyT, ValueT, KeyInfoT>::clear() [with KeyT = llvm::MachineBasicBlock*, ValueT = <unnamed>::BlockChain*, KeyInfoT = llvm::DenseMapInfo<llvm::MachineBasicBlock*>]: Assertion `NumEntries == 0 && "Node count imbalance!"' failed. Program received signal SIGABRT, Aborted. 0x00007ffff785c9...

...call of overloaded `AddInteger(uint32_t)' is ambiguous .../lib/CodeGen/SelectionDAG/SelectionDAG.cpp:1429: error: no matching function for call to `max(long unsigned int, unsigned int&)' .../include/llvm/ADT/DenseMap.h: In member function `void llvm::DenseMapIterator<KeyT, ValueT, KeyInfoT, ValueInfoT>::AdvancePastEmptyBuckets() [with KeyT = uint32_t, ValueT = llvm::Value*, KeyInfoT = llvm::DenseMapInfo<uint32_t>, ValueInfoT = llvm::DenseMapInfo<llvm::Value*>]': .../include/llvm/ADT/DenseMap.h:479: instantiated from `llvm::DenseMapIterator<KeyT, ValueT, KeyInf...

...itry N. Mikushin <maemarcus at gmail.com>: > Hi, > > After building out project in release mode, caught an assertion, which > we have not seen before: > > hello_f: /tmp/rpmbuild_debug/BUILD/llvm/build/include/llvm/ADT/DenseMap.h:126: > void llvm::DenseMap<KeyT, ValueT, KeyInfoT>::clear() [with KeyT = > llvm::MachineBasicBlock*, ValueT = <unnamed>::BlockChain*, KeyInfoT = > llvm::DenseMapInfo<llvm::MachineBasicBlock*>]: Assertion `NumEntries > == 0 && "Node count imbalance!"' failed. > > Program received signal SIGABRT, A...

...us at gmail.com>: >> Hi, >> >> After building out project in release mode, caught an assertion, which >> we have not seen before: >> >> hello_f: /tmp/rpmbuild_debug/BUILD/llvm/build/include/llvm/ADT/DenseMap.h:126: >> void llvm::DenseMap<KeyT, ValueT, KeyInfoT>::clear() [with KeyT = >> llvm::MachineBasicBlock*, ValueT = <unnamed>::BlockChain*, KeyInfoT = >> llvm::DenseMapInfo<llvm::MachineBasicBlock*>]: Assertion `NumEntries >> == 0 && "Node count imbalance!"' failed. >> >> Program receiv...

...seMapInfo<const llvm::Value *>, llvm::detail::DenseMapPair<const llvm::Value *, llvm::StringMapEntry<llvm::Value *> *>, false>::operator!=(const llvm::DenseMapIterator::ConstIterator &) const [KeyT = const llvm::Value *, ValueT = llvm::StringMapEntry<llvm::Value *> *, KeyInfoT = llvm::DenseMapInfo<const llvm::Value *>, Bucket = llvm::detail::DenseMapPair<const llvm::Value *, llvm::StringMapEntry<llvm::Value *> *>, IsConst = false]: Assertion `(!Ptr || isHandleInSync()) && "handle not in sync!"' failed. It seems like the given asse...

...t/type_traits.h. In > C++0x, this is named "conditional" (section 20.6.7), so I think you > should use the same name, despite the standard committee's bad taste. > > + DenseMapIterator(const DenseMapIterator<KeyT, ValueT, > + KeyInfoT, ValueInfoT, false>& I) > > This looks like it will make it impossible to copy const_iterators. I > guess it doesn't because the copy-constructor is auto-generated, but > please comment that and add tests for it and for the non-const->const > conversion to unittests/ADT...

...is named "conditional" (section 20.6.7), so I think you > >> should use the same name, despite the standard committee's bad taste. > >> > >> + DenseMapIterator(const DenseMapIterator<KeyT, ValueT, > >> + KeyInfoT, ValueInfoT, false>& > >> I) > >> > >> This looks like it will make it impossible to copy const_iterators. I > >> guess it doesn't because the copy-constructor is auto-generated, but > >> please comment that and add tests for it and for the no...

...into include/llvm/Support/type_traits.h. In C++0x, this is named "conditional" (section 20.6.7), so I think you should use the same name, despite the standard committee's bad taste. + DenseMapIterator(const DenseMapIterator<KeyT, ValueT, + KeyInfoT, ValueInfoT, false>& I) This looks like it will make it impossible to copy const_iterators. I guess it doesn't because the copy-constructor is auto-generated, but please comment that and add tests for it and for the non-const->const conversion to unittests/ADT/DenseMapTest.cpp. Othe...

...>> C++0x, this is named "conditional" (section 20.6.7), so I think you >> should use the same name, despite the standard committee's bad taste. >> >> +  DenseMapIterator(const DenseMapIterator<KeyT, ValueT, >> +                                          KeyInfoT, ValueInfoT, false>& >> I) >> >> This looks like it will make it impossible to copy const_iterators. I >> guess it doesn't because the copy-constructor is auto-generated, but >> please comment that and add tests for it and for the non-const->const >&gt...

Dear all, The first of the proposed patches (DenseMapIterator.patch) forbids implicit conversion of DenseMap::const_iterator to DenseMap::iterator which was possible because DenseMapIterator inherited (publicly) from DenseMapConstIterator. Conversion the other way around is now allowed as one may expect. The template DenseMapConstIterator is removed and the template parameter IsConst which

...conditional" (section 20.6.7), so I think you >> >> should use the same name, despite the standard committee's bad taste. >> >> >> >> + DenseMapIterator(const DenseMapIterator<KeyT, ValueT, >> >> + KeyInfoT, ValueInfoT, >> false>& >> >> I) >> >> >> >> This looks like it will make it impossible to copy const_iterators. I >> >> guess it doesn't because the copy-constructor is auto-generated, but >> >> please comment that and ad...

...this->grow(NumBuckets * 2); LookupBucketFor(Key, TheBucket); } + if (NumBuckets-(NumEntries+NumTombstones) < NumBuckets/8) { + this->grow(NumBuckets); + LookupBucketFor(Key, TheBucket); + } // If we are writing over a tombstone, remember this. if (!KeyInfoT::isEqual(TheBucket->first, getEmptyKey())) -- 1.7.4.1

...tries; if (NumEntries*4 >= NumBuckets*3 || NumBuckets-(NumEntries+NumTombstones) < NumBuckets/8) { this->grow(NumBuckets * 2); LookupBucketFor(Key, TheBucket); } - ++NumEntries; // If we are writing over a tombstone, remember this. if (!KeyInfoT::isEqual(TheBucket->first, getEmptyKey())) ------------------------------------------------------ Question: How can I test this? I was able to create a nifty Googletest unit test: Index: llvm.test/unittests/ADT/DenseSetTest.cpp =============================================================...

This series of patches address several issues causing memory usage to grow indefinetely on a long lived process. These are not convenional leaks -- memory would have been freed when the LLVM context or/and JIT engine is destroyed -- but for as long as they aren't the memory is usage effectively ubounded. The issues were found using valgrind with '--show-reachable=yes' option: 1.