[compiler-rt] [compiler-rt][ctx_instr] Add `ctx_profile` component (PR #89304)

Mircea Trofin via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 18:51:50 PDT 2024


mtrofin wrote:

> ```
> -  static void freeArenaList(Arena*& A);
> +  static void freeArenaList(Arena *&A);
>  
>    uint64_t size() const { return Size; }
>  
> @@ -51,5 +51,5 @@ private:
>    const uint64_t Size;
>  };
>  
> -}
> +} // namespace __ctx_profile
>  #endif // CTX_PROFILE_CTXINSTRPROFILING_H_
> diff --git a/compiler-rt/lib/ctx_profile/tests/CtxInstrProfilingTest.cpp b/compiler-rt/lib/ctx_profile/tests/CtxInstrProfilingTest.cpp
> index 1e7ddfadd8..44f[37](https://github.com/llvm/llvm-project/actions/runs/8744371029/job/23997070812?pr=89304#step:10:38)d2576 100644
> --- a/compiler-rt/lib/ctx_profile/tests/CtxInstrProfilingTest.cpp
> +++ b/compiler-rt/lib/ctx_profile/tests/CtxInstrProfilingTest.cpp
> @@ -4,11 +4,11 @@
>  using namespace __ctx_profile;
>  
>  TEST(ArenaTest, Basic) {
> -  Arena * A = Arena::allocateNewArena(1024);
> +  Arena *A = Arena::allocateNewArena(1024);
>    EXPECT_EQ(A->size(), 1024U);
>    EXPECT_EQ(A->next(), nullptr);
>  
> -  auto *M1 = A->tryBumpAllocate(1020); 
> +  auto *M1 = A->tryBumpAllocate(1020);
>    EXPECT_NE(M1, nullptr);
>    auto *M2 = A->tryBumpAllocate(4);
>    EXPECT_NE(M2, nullptr);
> 
> error: some formatters failed: clang-format
> ```

Ugh, forgot that - fixed. Thanks!

https://github.com/llvm/llvm-project/pull/89304


More information about the llvm-commits mailing list