[Lldb-commits] [lldb] [lldb] Remove pre GCC 4.8 workaround (PR #73981)

2023-11-30 Thread Brad Smith via lldb-commits
https://github.com/brad0 closed https://github.com/llvm/llvm-project/pull/73981 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Remove pre GCC 4.8 workaround (PR #73981)

2023-11-30 Thread Brad Smith via lldb-commits
brad0 wrote: > From the look of it, it looks like the minimum version of GCC that is > currently supported is 7.1. (See: > `llvm/cmake/modules/CheckCompilerVersion.cmake`) It was 7.1, it is now 7.4. https://reviews.llvm.org/D156286 https://github.com/llvm/llvm-project/pull/73981

[Lldb-commits] [lldb] [lldb] Remove pre GCC 4.8 workaround (PR #73981)

2023-11-30 Thread Alex Langford via lldb-commits
https://github.com/bulbazord approved this pull request. LGTM >From the look of it, it looks like the minimum version of GCC that is >currently supported is 7.1. (See: >`llvm/cmake/modules/CheckCompilerVersion.cmake`) https://github.com/llvm/llvm-project/pull/73981

[Lldb-commits] [lldb] [lldb] Remove pre GCC 4.8 workaround (PR #73981)

2023-11-30 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Brad Smith (brad0) Changes The minimum GCC version was bumped up from 4.8 to 5.1 and then even newer awhile ago so garbage collect the pre 4.8 workaround. https://reviews.llvm.org/D66188 --- Full diff:

[Lldb-commits] [lldb] [lldb] Remove pre GCC 4.8 workaround (PR #73981)

2023-11-30 Thread Brad Smith via lldb-commits
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/73981 The minimum GCC version was bumped up from 4.8 to 5.1 and then even newer awhile ago so garbage collect the pre 4.8 workaround. https://reviews.llvm.org/D66188 >From a79e4d06f16e2460c7521874c13370022bbb5952 Mon