[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-09 Thread George Burgess IV via cfe-commits
https://github.com/gburgessiv closed https://github.com/llvm/llvm-project/pull/111439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-09 Thread George Burgess IV via cfe-commits
gburgessiv wrote: (...Or a reviewer will find me - thank you!) https://github.com/llvm/llvm-project/pull/111439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-08 Thread Helena Kotas via cfe-commits
https://github.com/hekota approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/111439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-08 Thread George Burgess IV via cfe-commits
gburgessiv wrote: OK, thanks! I'll try to find someone with more context later today :) https://github.com/llvm/llvm-project/pull/111439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-07 Thread Damyan Pepper via cfe-commits
damyanp wrote: > Hey Damyan, GH suggested you as a reviewer for this. Would you be able to > TAL? Happy to find someone else if not :) I'm not sure why GH would have done that, I'm afraid I wouldn't be able to give an informed review here. (I might have a quick look anyway). https://github.co

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-07 Thread George Burgess IV via cfe-commits
gburgessiv wrote: Hey Damyan, GH suggested you as a reviewer for this. Would you be able to TAL? Happy to find someone else if not :) https://github.com/llvm/llvm-project/pull/111439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: George Burgess IV (gburgessiv) Changes At present, `__builtin_available` is really restrictive with its use. Overall, this seems like a good thing, since the analyses behind it are not very expensive. That said, it's very straightforward

[clang] [Sema] Support negation/parens with __builtin_available (PR #111439)

2024-10-07 Thread George Burgess IV via cfe-commits
https://github.com/gburgessiv created https://github.com/llvm/llvm-project/pull/111439 At present, `__builtin_available` is really restrictive with its use. Overall, this seems like a good thing, since the analyses behind it are not very expensive. That said, it's very straightforward to supp