================
@@ -0,0 +1,62 @@
+//===-- Definition of macros from endian.h 
--------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM 
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef __CLANG_ENDIAN_H
+#define __CLANG_ENDIAN_H
+
+// If the system has an endian.h, let's use that instead.
+#if __STDC_HOSTED__ && __has_include_next(<endian.h>)
----------------
daltenty wrote:

> These changes landed without a resolution to my question.

Sorry for missing that this was still outstanding.

>What is the justification for not including from the platform when in 
>freestanding specifically?

I think the rationale you two are outlining here makes sense, even if it is an 
edge case if the platform has the header available we may as well use it. 

Let me address removing this check as a follow on change, sorry for the 
confusion.


https://github.com/llvm/llvm-project/pull/186032
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to