[libcxx-commits] [PATCH] D129048: Rewording the "static_assert" to static assertion

Muhammad Usman Shahid via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jul 22 09:00:31 PDT 2022


Codesbyusman added a comment.

In D129048#3671657 <https://reviews.llvm.org/D129048#3671657>, @ldionne wrote:

> In D129048#3671568 <https://reviews.llvm.org/D129048#3671568>, @aaron.ballman wrote:
>
>> FWIW, I've convinced myself that I agree with you here that the burden probably should have been on libc++ maintainers in this case. libc++ almost feels more like a downstream consumer of Clang in terms of testing needs, so I think when tests break because Clang diagnostic wording or behavior changes in a standards conforming way, libc++ folks should address it, but if Clang changes in a way that's not standards conforming, then Clang folks should address it. (Roughly speaking.)
>
> Yes, exactly. It's always possible to write tests that depend on implementation details of another component in subtle ways, and the fact that such brittle tests exist doesn't create a responsibility on that component to avoid breaking those downstream users. Here, it's about Clang making a valid change and libc++ containing brittle tests, but it happens quite often where libc++ changes something valid and a downstream consumer is broken in some way. The LLVM revert culture has some benefits to keep things working in a post-commit CI world, however I've noticed that it also creates a lot of friction between projects and sometimes makes important work much more difficult to land than it should. For example, we're currently in the middle of D128146 <https://reviews.llvm.org/D128146> where LLDB reverted an important patch for LLVM 15 because one of their tests broke. This is not something that comes up super often, but it's extremely disruptive and frustrating when it does, and I think it would be worth trying to address. Anyway, I'm digressing now, but I'll try to talk to a few people at the LLVM Dev Meeting to see if this is a shared concern and to think about potential solutions to address this.
>
>> That said, I absolutely think we all need to continue to collaborate closely with one another as best we can when issues arise, and I really appreciate the discussion on how we can do that!
>
> Agreed.
>
>> For this particular issue, I'd like @Codesbyusman to continue to try to fix the libc++ testing issues (it's good experience), but if that takes significantly longer (say, more than 8 hours of his effort), perhaps @ldionne or someone else from libc++ will have a moment to step in to help?
>
> Replacing `static_assert` by `(static_assert|static assertion)` should do the trick. See the patch attached to this comment, I think it should satisfy the CI @Codesbyusman.
> F23872372: static_assert.diff <https://reviews.llvm.org/F23872372>

thanks


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129048/new/

https://reviews.llvm.org/D129048



More information about the libcxx-commits mailing list