From fbdf17ae6866b60540f41d7c4928f25b5f2a8419 Mon Sep 17 00:00:00 2001
From: Daniel Bertalan <dani@danielbertalan.dev>
Date: Sat, 14 Aug 2021 16:56:55 +0200
Subject: [PATCH] LibC: Don't flatten `malloc` and `free`

This is no longer needed as per the previous commit, UserspaceEmulator's
malloc tracer now correctly handles functions called from within
`malloc` and `free`. This might also have a benefit on performance
because forcibly inlining all function calls pessimizes cache locality.
---
 Userland/Libraries/LibC/malloc.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Userland/Libraries/LibC/malloc.cpp b/Userland/Libraries/LibC/malloc.cpp
index b41a704c94..502025ad38 100644
--- a/Userland/Libraries/LibC/malloc.cpp
+++ b/Userland/Libraries/LibC/malloc.cpp
@@ -410,7 +410,7 @@ static void free_impl(void* ptr)
     }
 }
 
-[[gnu::flatten]] void* malloc(size_t size)
+void* malloc(size_t size)
 {
     MemoryAuditingSuppressor suppressor;
     void* ptr = malloc_impl(size, CallerWillInitializeMemory::No);
@@ -419,7 +419,7 @@ static void free_impl(void* ptr)
     return ptr;
 }
 
-[[gnu::flatten]] void free(void* ptr)
+void free(void* ptr)
 {
     MemoryAuditingSuppressor suppressor;
     if (s_profiling)