philnik777 wrote:
Closing, since it seems to be abandoned.
https://github.com/llvm/llvm-project/pull/74803
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/74803
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/W-50243 updated
https://github.com/llvm/llvm-project/pull/74803
>From bcaaab24a927624669200113857fee842103d325 Mon Sep 17 00:00:00 2001
From: W-50243
Date: Thu, 7 Dec 2023 19:42:29 +0800
Subject: [PATCH 1/2] [Clang][Builtin] add __builtin_exit
---
clang/include/clang/Basic/
philnik777 wrote:
I'm not really convinced that `__builtin_exit` is widely used. If you filter
for C/C++ and exclude GCC there aren't many matches:
https://sourcegraph.com/search?q=context:global+__builtin_exit+count:all+-file:gcc/.*+%28lang:C+OR+lang:C%2B%2B%29&patternType=standard&sm=0&groupB
W-50243 wrote:
I added __builtin_exit() because __builtin_exit() was used in some projects
when I switched from GCC to LLVM. I don't think this is an isolated case,
because the __builtin_exit() function is still widely
used.[https://sourcegraph.com/search?q=context:global+__builtin_exit&patter
erichkeane wrote:
This comes down to a question of 'why' from me? Typically we add builtins like
this because libc++ need them, or libc/libstdc++ use them. In this case, I
don't see any evidence of anyone really needing it.
While GCC supports it, I don't really see value here?
Also, this is
https://github.com/W-50243 updated
https://github.com/llvm/llvm-project/pull/74803
>From bcaaab24a927624669200113857fee842103d325 Mon Sep 17 00:00:00 2001
From: W-50243
Date: Thu, 7 Dec 2023 19:42:29 +0800
Subject: [PATCH 1/2] [Clang][Builtin] add __builtin_exit
---
clang/include/clang/Basic/
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (W-50243)
Changes
Add __builtin_exit in `clang/include/clang/Basic/Builtins.def` . It works
perfectly on arm64be and arm64le.
---
Full diff: https://github.com/llvm/llvm-project/pull/74803.diff
2 Files Affected:
- (modified) clang
https://github.com/W-50243 created
https://github.com/llvm/llvm-project/pull/74803
Add __builtin_exit in `clang/include/clang/Basic/Builtins.def` . It works
perfectly on arm64be and arm64le.
>From bcaaab24a927624669200113857fee842103d325 Mon Sep 17 00:00:00 2001
From: W-50243
Date: Thu, 7 Dec
