[PATCH] D126550: Make report_fatal_error respect its GenCrashDiag argument so it doesn't generate a backtrace
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 29 10:45:19 PDT 2022
MaskRay added inline comments.
================
Comment at: llvm/lib/Support/ErrorHandling.cpp:125
+ else
+ exit(-1);
}
----------------
MaskRay wrote:
> https://www.opengroup.org/austin/docs/austin_1110.pdf
>
> "Calling exit() with a value greater than 255 or less than 0 is something that only programs
> which are specifically designed to have their exit status obtained by waitid() should do
> (since it does not truncate the exit status to 8 bits). ``Pure ISO Cāā programs that call
> exit(EXIT_FAILURE) do not meet this design criterion."
A value between 1 and 125 can be used.
1 and 2 are often used for other purposes.
Perhaps use a value larger than `EX__MAX` in sysexits.h
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126550/new/
https://reviews.llvm.org/D126550
More information about the llvm-commits
mailing list