[libcxx-commits] [libcxx] [libcxx] Add a missing include for __bit_iterator (PR #127015)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Mar 18 08:11:35 PDT 2025
ldionne wrote:
Now that we have a better understanding of what's going on, I'd like to echo what @philnik777 said in https://github.com/llvm/llvm-project/pull/127015#issuecomment-2655830924. We shouldn't fix things blindly by patching over them without understanding the root cause, especially when we do so without adding proper test coverage. I know those are just build issues so it may seem like a "simple hacky fix" is good enough, but it really isn't since this sort of technical debt compounds over years of development and gets us to a place where we don't understand our codebase anymore. Going forward, let's please try to go to the bottom of these kinds of issues, that's the only way we'll build something solid.
https://github.com/llvm/llvm-project/pull/127015
More information about the libcxx-commits
mailing list