Skip to content

Commit 2b7227d

Browse files
committed
[libc] Fix RPC server global after mass replace of __llvm_libc
Summary: This variable needs a reserved name starting with `__`. It was mistakenly changed with a mass replace. It happened to work because the tests still picked up the associated symbol, but it just became a bad name because it's not reserved anymore.
1 parent f8aab28 commit 2b7227d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

libc/src/__support/RPC/rpc_client.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ Client client;
1818
/// Externally visible symbol to signify the usage of an RPC client to
1919
/// whomever needs to run the server as well as provide a way to initialize
2020
/// the client with a copy..
21-
extern "C" [[gnu::visibility("protected")]] void *LIBC_NAMESPACE_rpc_client =
21+
extern "C" [[gnu::visibility("protected")]] const void *__llvm_libc_rpc_client =
2222
&client;
2323

2424
} // namespace rpc

libc/utils/gpu/server/rpc_server.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ const uint64_t RPC_MAXIMUM_PORT_COUNT = 512;
2222

2323
/// The symbol name associated with the client for use with the LLVM C library
2424
/// implementation.
25-
const char *const rpc_client_symbol_name = "LIBC_NAMESPACE_rpc_client";
25+
const char *const rpc_client_symbol_name = "__llvm_libc_rpc_client";
2626

2727
/// status codes.
2828
typedef enum {

0 commit comments

Comments
 (0)