Skip to content

Commit

Permalink
Don't inline get[S|U]LEB128Size() until they are proved to be hot.
Browse files Browse the repository at this point in the history
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@201939 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
loganchien committed Feb 22, 2014
1 parent 80668d1 commit 3c288fc
Show file tree
Hide file tree
Showing 3 changed files with 47 additions and 21 deletions.
23 changes: 2 additions & 21 deletions include/llvm/Support/LEB128.h
Original file line number Diff line number Diff line change
Expand Up @@ -91,29 +91,10 @@ inline uint64_t decodeULEB128(const uint8_t *p, unsigned *n = 0) {
}

/// Utility function to get the size of the ULEB128-encoded value.
inline unsigned getULEB128Size(uint64_t Value) {
unsigned Size = 0;
do {
Value >>= 7;
Size += sizeof(int8_t);
} while (Value);
return Size;
}
extern unsigned getULEB128Size(uint64_t Value);

/// Utility function to get the size of the SLEB128-encoded value.
inline unsigned getSLEB128Size(int64_t Value) {
unsigned Size = 0;
int Sign = Value >> (8 * sizeof(Value) - 1);
bool IsMore;

do {
unsigned Byte = Value & 0x7f;
Value >>= 7;
IsMore = Value != Sign || ((Byte ^ Sign) & 0x40) != 0;
Size += sizeof(int8_t);
} while (IsMore);
return Size;
}
extern unsigned getSLEB128Size(int64_t Value);

} // namespace llvm

Expand Down
1 change: 1 addition & 0 deletions lib/Support/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ add_llvm_library(LLVMSupport
IntrusiveRefCntPtr.cpp
IsInf.cpp
IsNAN.cpp
LEB128.cpp
LineIterator.cpp
Locale.cpp
LockFileManager.cpp
Expand Down
44 changes: 44 additions & 0 deletions lib/Support/LEB128.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
//===- LEB128.cpp - LEB128 utility functions implementation -----*- C++ -*-===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
//
// This file implements some utility functions for encoding SLEB128 and
// ULEB128 values.
//
//===----------------------------------------------------------------------===//

#include "llvm/Support/LEB128.h"

namespace llvm {

/// Utility function to get the size of the ULEB128-encoded value.
unsigned getULEB128Size(uint64_t Value) {
unsigned Size = 0;
do {
Value >>= 7;
Size += sizeof(int8_t);
} while (Value);
return Size;
}

/// Utility function to get the size of the SLEB128-encoded value.
unsigned getSLEB128Size(int64_t Value) {
unsigned Size = 0;
int Sign = Value >> (8 * sizeof(Value) - 1);
bool IsMore;

do {
unsigned Byte = Value & 0x7f;
Value >>= 7;
IsMore = Value != Sign || ((Byte ^ Sign) & 0x40) != 0;
Size += sizeof(int8_t);
} while (IsMore);
return Size;
}

} // namespace llvm

0 comments on commit 3c288fc

Please sign in to comment.