Changes in IntelJITEventListener - By Arch Robinson
authorElena Demikhovsky <elena.demikhovsky@intel.com>
Sun, 6 Apr 2014 11:08:33 +0000 (11:08 +0000)
committerElena Demikhovsky <elena.demikhovsky@intel.com>
Sun, 6 Apr 2014 11:08:33 +0000 (11:08 +0000)
commitec8891fddecf0e5441bc20222e47dc571afe6fc5
tree9cf0a01937b3958f7436b2628f85a8b90ce7d702
parent2e858fcaad72d7b6ed994436e25af083ee5ecde2
Changes in IntelJITEventListener - By Arch Robinson
   - take->release: LLVM has moved to C++11.  MockWrapper became an instance of unique_ptr.

   - method symbol_iterator::increment disappeared recently, in this revision:

     r200442 | rafael | 2014-01-29 20:49:50 -0600 (Wed, 29 Jan 2014) | 9 lines

Simplify the handling of iterators in ObjectFile.

None of the object file formats reported error on iterator increment. In
retrospect, that is not too surprising: no object format stores symbols or
sections in a linked list or other structure that requires chasing pointers.
As a consequence, all error checking can be done on begin() and end().

This reduces the text segment of bin/llvm-readobj in my machine from 521233 to
518526 bytes.

My change mimics the change that the revision made to lib/DebugInfo/DWARFContext.cpp .

    - const_cast: Shut up a warning from gcc.

I ran unittests/ExecutionEngine/JIT/Debug+Asserts/JITTests to make sure it worked.

- Arch

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@205689 91177308-0d34-0410-b5e6-96231b3b80d8
lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp
unittests/ExecutionEngine/JIT/IntelJITEventListenerTest.cpp