From 6273b6d9dcc10b47570e260facad795c7afe87e8 Mon Sep 17 00:00:00 2001
From: Joseph Huber <35342157+jhuber6@users.noreply.github.com>
Date: Tue, 26 Sep 2023 15:24:28 -0500
Subject: [PATCH] [libc] Change RPC opcode enum definition (#67439)

Summary:
This enum previously manually specified the value. This just made it
unnecessarily difficult to add new ones without changing everything.
This patch also makes it compatible with C by removing the `:`
annotation and instead using the `LAST` method.
---
 libc/include/llvm-libc-types/rpc_opcodes_t.h | 39 ++++++++++----------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/libc/include/llvm-libc-types/rpc_opcodes_t.h b/libc/include/llvm-libc-types/rpc_opcodes_t.h
index 77fa70946697..61e17756fa64 100644
--- a/libc/include/llvm-libc-types/rpc_opcodes_t.h
+++ b/libc/include/llvm-libc-types/rpc_opcodes_t.h
@@ -9,26 +9,27 @@
 #ifndef __LLVM_LIBC_TYPES_RPC_OPCODE_H__
 #define __LLVM_LIBC_TYPES_RPC_OPCODE_H__
 
-typedef enum : unsigned short {
+typedef enum {
   RPC_NOOP = 0,
-  RPC_EXIT = 1,
-  RPC_WRITE_TO_STDOUT = 2,
-  RPC_WRITE_TO_STDERR = 3,
-  RPC_WRITE_TO_STREAM = 4,
-  RPC_WRITE_TO_STDOUT_NEWLINE = 5,
-  RPC_READ_FROM_STREAM = 6,
-  RPC_OPEN_FILE = 7,
-  RPC_CLOSE_FILE = 8,
-  RPC_MALLOC = 9,
-  RPC_FREE = 10,
-  RPC_HOST_CALL = 11,
-  RPC_ABORT = 12,
-  RPC_FEOF = 13,
-  RPC_FERROR = 14,
-  RPC_CLEARERR = 15,
-  RPC_FSEEK = 16,
-  RPC_FTELL = 17,
-  RPC_FFLUSH = 18,
+  RPC_EXIT,
+  RPC_WRITE_TO_STDOUT,
+  RPC_WRITE_TO_STDERR,
+  RPC_WRITE_TO_STREAM,
+  RPC_WRITE_TO_STDOUT_NEWLINE,
+  RPC_READ_FROM_STREAM,
+  RPC_OPEN_FILE,
+  RPC_CLOSE_FILE,
+  RPC_MALLOC,
+  RPC_FREE,
+  RPC_HOST_CALL,
+  RPC_ABORT,
+  RPC_FEOF,
+  RPC_FERROR,
+  RPC_CLEARERR,
+  RPC_FSEEK,
+  RPC_FTELL,
+  RPC_FFLUSH,
+  RPC_LAST = 0xFFFF,
 } rpc_opcode_t;
 
 #endif // __LLVM_LIBC_TYPES_RPC_OPCODE_H__