[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-26 Thread Sean Perry via cfe-commits
https://github.com/perry-ca closed https://github.com/llvm/llvm-project/pull/109852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-26 Thread Sean Perry via cfe-commits
perry-ca wrote: Thanks for the offer David. I do have permission now. I held off merging yesterday because the builds were all broken from some other change. The builds all pass now. I'll merge it. https://github.com/llvm/llvm-project/pull/109852 ___

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-26 Thread David Spickett via cfe-commits
DavidSpickett wrote: Do you have merge permissions? I can merge it for you if not. https://github.com/llvm/llvm-project/pull/109852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-26 Thread Sean Perry via cfe-commits
https://github.com/perry-ca updated https://github.com/llvm/llvm-project/pull/109852 >From f8bb9d99ae9b9e8fd546d8a5a801681c721cb11d Mon Sep 17 00:00:00 2001 From: Sean Perry Date: Fri, 13 Sep 2024 16:49:02 -0500 Subject: [PATCH] use errc instead of hard coded text for messages --- clang/test/

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-25 Thread David Spickett via cfe-commits
https://github.com/DavidSpickett approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/109852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-25 Thread David Spickett via cfe-commits
DavidSpickett wrote: CI failure is: ``` _bk;t=1727216861348Timed Out Tests (1): _bk;t=1727216861348 lldb-api :: functionalities/fork/concurrent_vfork/TestConcurrentVFork.py ``` Ignore that it's due to https://github.com/llvm/llvm-project/issues/104609. https://github.com/llvm/llvm-project

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-25 Thread David Spickett via cfe-commits
DavidSpickett wrote: I think https://github.com/llvm/llvm-project/blob/5a191e3cd904b5fee202569e54fe04f91f1a697f/llvm/utils/lit/lit/llvm/config.py#L411 works such that if the messages cannot be found (cross compiling and such) we default to asking Python what it thinks they should be. I wonder

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-24 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-debuginfo Author: Sean Perry (perry-ca) Changes Several lit tests look for messages with text generated from strerror() such as "no such file or directory". The value can change slightly from system to system. Use the llvm-lit macro `%errc_`

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-24 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Sean Perry (perry-ca) Changes Several lit tests look for messages with text generated from strerror() such as "no such file or directory". The value can change slightly from system to system. Use the llvm-lit macro `%errc_` ins

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-24 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-driver Author: Sean Perry (perry-ca) Changes Several lit tests look for messages with text generated from strerror() such as "no such file or directory". The value can change slightly from system to system. Use the llvm-lit macro `%errc_

[clang] [llvm] Use %errc to get text for system errors (PR #109852)

2024-09-24 Thread Sean Perry via cfe-commits
https://github.com/perry-ca created https://github.com/llvm/llvm-project/pull/109852 Several lit tests look for messages with text generated from strerror() such as "no such file or directory". The value can change slightly from system to system. Use the llvm-lit macro `%errc_` instead. Thi