[Lldb-commits] [lldb] [lldb] Print a warning on checksum mismatch (PR #107968)
Martin Storsjö via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 12 10:18:34 PDT 2024
mstorsjo wrote:
> This broke building in mingw configurations, and possibly a few others as well.
>
> The `ReportWarning` function takes a pointer to a `std::once_flag`, while this is passing it with a `llvm::once_flag`. In many configurations, these are the same type, but in a couple ones, they're not.
>
> LLDB seems to be using a mixture of `std::once_flag` and `llvm::once_flag` throughout, with nontrivial numbers of both, so there's no one obvious preferred way to go here... Or should `ReportWarning` be extended to accept either kinds of flags, until the implementation is settled on either of them?
If there are no suggestions on a way to go here, I'll push a fix within a couple hours, that fixes compilation - probably switching the newly added code to `std::once_flag` as that's what the existing function takes.
https://github.com/llvm/llvm-project/pull/107968
More information about the lldb-commits
mailing list