[llvm] r234404 - [jitlistener] Remove unused code

Lang Hames lhames at gmail.com
Wed Apr 8 12:01:46 PDT 2015


Cool! Thanks Benjamin.

- Lang.

On Wed, Apr 8, 2015 at 6:17 AM, Benjamin Kramer <benny.kra at googlemail.com>
wrote:

> Author: d0k
> Date: Wed Apr  8 08:17:48 2015
> New Revision: 234404
>
> URL: http://llvm.org/viewvc/llvm-project?rev=234404&view=rev
> Log:
> [jitlistener] Remove unused code
>
> Removed:
>     llvm/trunk/lib/ExecutionEngine/EventListenerCommon.h
> Modified:
>     llvm/trunk/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp
>     llvm/trunk/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp
>
> Removed: llvm/trunk/lib/ExecutionEngine/EventListenerCommon.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/EventListenerCommon.h?rev=234403&view=auto
>
> ==============================================================================
> --- llvm/trunk/lib/ExecutionEngine/EventListenerCommon.h (original)
> +++ llvm/trunk/lib/ExecutionEngine/EventListenerCommon.h (removed)
> @@ -1,68 +0,0 @@
> -//===-- JIT.h - Abstract Execution Engine Interface -------------*- C++
> -*-===//
> -//
> -//                     The LLVM Compiler Infrastructure
> -//
> -// This file is distributed under the University of Illinois Open Source
> -// License. See LICENSE.TXT for details.
> -//
>
> -//===----------------------------------------------------------------------===//
> -//
> -// Common functionality for JITEventListener implementations
> -//
>
> -//===----------------------------------------------------------------------===//
> -
> -#ifndef EVENT_LISTENER_COMMON_H
> -#define EVENT_LISTENER_COMMON_H
> -
> -#include "llvm/ADT/DenseMap.h"
> -#include "llvm/IR/DebugInfo.h"
> -#include "llvm/IR/Metadata.h"
> -#include "llvm/IR/ValueHandle.h"
> -#include "llvm/Support/Path.h"
> -
> -namespace llvm {
> -
> -namespace jitprofiling {
> -
> -class FilenameCache {
> -  // Holds the filename of each Scope, so that we can pass a
> null-terminated
> -  // string into oprofile.
> -  DenseMap<const MDNode *, std::string> Filenames;
> -  DenseMap<const MDNode *, std::string> Paths;
> -
> - public:
> -  const char *getFilename(MDNode *Scope) {
> -    assert(Scope->isResolved() && "Expected Scope to be resolved");
> -    std::string &Filename = Filenames[Scope];
> -    if (Filename.empty()) {
> -      DIScope DIScope(Scope);
> -      Filename = DIScope.getFilename();
> -    }
> -    return Filename.c_str();
> -  }
> -
> -  const char *getFullPath(MDNode *Scope) {
> -    assert(Scope->isResolved() && "Expected Scope to be resolved");
> -    std::string &P = Paths[Scope];
> -    if (P.empty()) {
> -      DIScope DIScope(Scope);
> -      StringRef DirName = DIScope.getDirectory();
> -      StringRef FileName = DIScope.getFilename();
> -      SmallString<256> FullPath;
> -      if (DirName != "." && DirName != "") {
> -        FullPath = DirName;
> -      }
> -      if (FileName != "") {
> -        sys::path::append(FullPath, FileName);
> -      }
> -      P = FullPath.str();
> -    }
> -    return P.c_str();
> -  }
> -};
> -
> -} // namespace jitprofiling
> -
> -} // namespace llvm
> -
> -#endif //EVENT_LISTENER_COMMON_H
>
> Modified:
> llvm/trunk/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp?rev=234404&r1=234403&r2=234404&view=diff
>
> ==============================================================================
> ---
> llvm/trunk/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp
> (original)
> +++
> llvm/trunk/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp Wed
> Apr  8 08:17:48 2015
> @@ -13,7 +13,6 @@
>
>  //===----------------------------------------------------------------------===//
>
>  #include "llvm/Config/config.h"
> -#include "EventListenerCommon.h"
>  #include "IntelJITEventsWrapper.h"
>  #include "llvm/ADT/DenseMap.h"
>  #include "llvm/CodeGen/MachineFunction.h"
> @@ -29,7 +28,6 @@
>  #include "llvm/Support/raw_ostream.h"
>
>  using namespace llvm;
> -using namespace llvm::jitprofiling;
>  using namespace llvm::object;
>
>  #define DEBUG_TYPE "amplifier-jit-event-listener"
> @@ -41,7 +39,6 @@ class IntelJITEventListener : public JIT
>
>    std::unique_ptr<IntelJITEventsWrapper> Wrapper;
>    MethodIDMap MethodIDs;
> -  FilenameCache Filenames;
>
>    typedef SmallVector<const void *, 64> MethodAddressVector;
>    typedef DenseMap<const void *, MethodAddressVector>  ObjectMap;
>
> Modified:
> llvm/trunk/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp?rev=234404&r1=234403&r2=234404&view=diff
>
> ==============================================================================
> ---
> llvm/trunk/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp
> (original)
> +++
> llvm/trunk/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp Wed
> Apr  8 08:17:48 2015
> @@ -13,7 +13,6 @@
>
>  //===----------------------------------------------------------------------===//
>
>  #include "llvm/Config/config.h"
> -#include "EventListenerCommon.h"
>  #include "llvm/CodeGen/MachineFunction.h"
>  #include "llvm/ExecutionEngine/JITEventListener.h"
>  #include "llvm/ExecutionEngine/OProfileWrapper.h"
> @@ -28,7 +27,6 @@
>  #include <fcntl.h>
>
>  using namespace llvm;
> -using namespace llvm::jitprofiling;
>  using namespace llvm::object;
>
>  #define DEBUG_TYPE "oprofile-jit-event-listener"
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150408/8731c82f/attachment.html>


More information about the llvm-commits mailing list