[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

logan-5 added a comment. In D84126#2171469 , @labath wrote: > Instead of adding -Wno-suggest-override to every user it might be cleaner to > add the flag to the INTERFACE_COMPILE_OPTIONS property of the relevant > targets (gtest, gmock, and gbenchmark ?)

[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

logan-5 added a comment. In D84126#2165585 , @nemanjai wrote: > There is a lot of instances where this warning fires in the unit tests since > gtest does not appear to use the `override` keyword. This in turn causes > failures on bootstrap builds that us

[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

logan-5 added a comment. In D84126#2166952 , @erichkeane wrote: > I'd suggest disabling this unless the GCC version is >9.1. I've got a patch addressing exactly this issue here, hoping to get it landed soon: https://reviews.llvm.org/D84292 Thanks again

[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

logan-5 created this revision. logan-5 added reviewers: dblaikie, rnk, aaron.ballman, ldionne. Herald added subscribers: llvm-commits, libcxx-commits, lldb-commits, lebedev.ri, dexonsmith, mgorny. Herald added a reviewer: lebedev.ri. Herald added projects: LLDB, libc++, libc++abi, LLVM. Herald add

[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

logan-5 added a comment. All the failures related to this patch are now resolved as far as I can tell. I'm very sorry for the trouble. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84126/new/ https://reviews.llvm.org/D84126

[Lldb-commits] [PATCH] D84126: Enable -Wsuggest-override in the LLVM build

This revision was automatically updated to reflect the committed changes. Closed by commit rG8b16e45f66e2: Enable -Wsuggest-override in the LLVM build (authored by logan-5). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84126/new/ https://reviews.ll

[Lldb-commits] [PATCH] D83847: [lldb][NFC] Add 'override' where missing in source/ and tools/

This revision was automatically updated to reflect the committed changes. Closed by commit rG44b43a52dc17: [lldb][NFC] Add 'override' where missing in source/ and tools/ (authored by logan-5). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83847/new/

[Lldb-commits] [PATCH] D83847: [lldb][NFC] Add 'override' where missing in source/ and tools/

logan-5 created this revision. logan-5 added a reviewer: clayborg. logan-5 added a project: LLDB. Herald added a subscriber: lldb-commits. These were found by Clang's new `-Wsuggest-override`. This patch doesn't touch any code in unittests/, since much of it intentionally doesn't use `override`