Skip to content

Commit b5dbce3

Browse files
[libc][uefi] review adjustments
1 parent 3aaccf0 commit b5dbce3

File tree

5 files changed

+14
-25
lines changed

5 files changed

+14
-25
lines changed

libc/include/Uefi.h.def

Lines changed: 0 additions & 16 deletions
This file was deleted.

libc/src/__support/OSUtil/uefi/error.h

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,15 @@
1818

1919
namespace LIBC_NAMESPACE_DECL {
2020

21-
#define EFI_ERROR_MAX_BIT (cpp::numeric_limits<EFI_STATUS>::max())
22-
#define EFI_ENCODE_ERROR(value) \
23-
(EFI_ERROR_MAX_BIT | (EFI_ERROR_MAX_BIT >> 2) | (value))
24-
#define EFI_ENCODE_WARNING(value) ((EFI_ERROR_MAX_BIT >> 2) | (value))
21+
static constexpr int EFI_ERROR_MAX_BIT = cpp::numeric_limits<EFI_STATUS>::max();
22+
23+
static constexpr int EFI_ENCODE_ERROR(int value) {
24+
return EFI_ERROR_MAX_BIT | (EFI_ERROR_MAX_BIT >> 2) | (value);
25+
}
26+
27+
static constexpr int EFI_ENCODE_WARNING(int value) {
28+
return (EFI_ERROR_MAX_BIT >> 2) | (value);
29+
}
2530

2631
struct UefiStatusErrnoEntry {
2732
EFI_STATUS status;
@@ -79,7 +84,7 @@ LIBC_INLINE int uefi_status_to_errno(EFI_STATUS status) {
7984
}
8085

8186
// Unknown type
82-
__builtin_unreachable();
87+
return EINVAL;
8388
}
8489

8590
LIBC_INLINE EFI_STATUS errno_to_uefi_status(int errno_value) {
@@ -91,7 +96,7 @@ LIBC_INLINE EFI_STATUS errno_to_uefi_status(int errno_value) {
9196
}
9297

9398
// Unknown type
94-
__builtin_unreachable();
99+
return EFI_INVALID_PARAMETER;
95100
}
96101

97102
} // namespace LIBC_NAMESPACE_DECL

libc/src/__support/OSUtil/uefi/exit.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
//===-----------------------------------------------------------------===//
88

99
#include "src/__support/OSUtil/exit.h"
10-
#include "config/uefi/app.h"
10+
#include "config/uefi.h"
1111
#include "include/llvm-libc-types/EFI_SYSTEM_TABLE.h"
1212
#include "src/__support/macros/config.h"
1313

libc/src/__support/OSUtil/uefi/io.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
#include "io.h"
1010

1111
#include "Uefi.h"
12-
#include "config/uefi/app.h"
12+
#include "config/app.h"
1313
#include "src/__support/CPP/string_view.h"
1414
#include "src/__support/macros/config.h"
1515

libc/startup/uefi/crt1.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88

9-
#include "config/uefi/app.h"
9+
#include "config/app.h"
1010
#include "include/llvm-libc-types/EFI_STATUS.h"
1111
#include "src/__support/OSUtil/uefi/error.h"
1212
#include "src/__support/macros/config.h"

0 commit comments

Comments
 (0)