[PATCH] Fix jit event listeners

[Andy Kaylor]

Hi lhames,

This patch fixes a problem that had crept in with the JIT event listeners (both the oprofile listener and the Intel-specific listener) and replaces the tests, which had rotted from disuse.

Arch Robison contributed the fix to EventListenerCommon.h and handed it off to me since I was working on fixing the JitListener tests.

The new JitListener tests are fairly basic.  It would be nice to have a test for the inlined function case, but I wanted to just get something working for now.  I'm not sure that the old parameter and common symbol tests ever had any relevance to the JitListeners, which only report functions and line numbers.

I have initiated a request to have the LLVM_USE_INTEL_JITEVENTS option turned on for one of the buildbots so that these tests will be run regularly.  The people who own the buildbot I have in mind are currently on holiday, but they did tell me that they are open to doing this.

Long term I would like to enable both the oprofile and the Intel JIT listeners by default for applicable platforms (removing the code that automatically registers the listeners) so that more developers will see emerging failures before the commits that cause them.  I don't think there's anything architecture-specific in the tests (they don't even depend on the profiling tools being present) so it might be possible to enable the tests everywhere.  It would also be nice to extend the current llvm-jitlistener tool so that it is also capable of testing the oprofile listener (which currently has no tests).  At some point I'd also like to refactor the oprofile and Intel listeners so that they share the code to parse debug info.

REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6905

Files:
  lib/ExecutionEngine/EventListenerCommon.h
  test/JitListener/lit.local.cfg
  test/JitListener/multiple.ll
  test/JitListener/simple.ll
  test/JitListener/test-common-symbols.ll
  test/JitListener/test-inline.ll
  test/JitListener/test-parameters.ll
  tools/llvm-jitlistener/llvm-jitlistener.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6905.17943.patch
Type: text/x-patch
Size: 48754 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150109/81453967/attachment.bin>