From 3c4b3a54581fe5badf08b9a0a32f8b2e5f55151d Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Wed, 30 Dec 2015 23:37:25 +0000 Subject: [PATCH] Revert "Revert "[ptr-traits] Implement the base pointer traits using the actual"" This reverts commit r256642 and restores r256620 now that Tobias has updated Polly. There are still some potential problems with the code in Polly that I've sent post-commit review about, but they're unlikely to break anything in practice, and I'd like to avoid the rest of LLVM and Clang regressing here. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@256656 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Support/PointerLikeTypeTraits.h | 28 ++++++++++++++++---- 1 file changed, 23 insertions(+), 5 deletions(-) diff --git a/include/llvm/Support/PointerLikeTypeTraits.h b/include/llvm/Support/PointerLikeTypeTraits.h index 95d735869f2..c12d237b279 100644 --- a/include/llvm/Support/PointerLikeTypeTraits.h +++ b/include/llvm/Support/PointerLikeTypeTraits.h @@ -15,6 +15,7 @@ #ifndef LLVM_SUPPORT_POINTERLIKETYPETRAITS_H #define LLVM_SUPPORT_POINTERLIKETYPETRAITS_H +#include "llvm/Support/AlignOf.h" #include "llvm/Support/DataTypes.h" namespace llvm { @@ -27,15 +28,32 @@ template class PointerLikeTypeTraits { // getNumLowBitsAvailable }; +namespace detail { +/// A tiny meta function to compute the log2 of a compile time constant. +template +struct ConstantLog2 + : std::integral_constant::value + 1> {}; +template <> struct ConstantLog2<1> : std::integral_constant {}; +} + // Provide PointerLikeTypeTraits for non-cvr pointers. -template class PointerLikeTypeTraits { -public: +template struct PointerLikeTypeTraits { static inline void *getAsVoidPointer(T *P) { return P; } static inline T *getFromVoidPointer(void *P) { return static_cast(P); } - /// Note, we assume here that malloc returns objects at least 4-byte aligned. - /// However, this may be wrong, or pointers may be from something other than - /// malloc. In this case, you should specialize this template to reduce this. + enum { + NumLowBitsAvailable = detail::ConstantLog2::Alignment>::value + }; +}; + +template <> struct PointerLikeTypeTraits { + static inline void *getAsVoidPointer(void *P) { return P; } + static inline void *getFromVoidPointer(void *P) { return P; } + + /// Note, we assume here that void* is related to raw malloc'ed memory and + /// that malloc returns objects at least 4-byte aligned. However, this may be + /// wrong, or pointers may be from something other than malloc. In this case, + /// you should specify a real typed pointer or avoid this template. /// /// All clients should use assertions to do a run-time check to ensure that /// this is actually true. -- 2.34.1