katzdm wrote:
> @katzdm Does it make sense to file an issue so we don't forget to bring the
> warning back?
Yep, for sure - I've opened #92656.
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
Fznamznon wrote:
@katzdm
Does it make sense to file an issue so we don't forget to bring the warning
back?
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
efriedma-quic wrote:
clang already generates certain diagnostics from ExprConstant; expanding the
set of diagnostics could be reasonable.
Alternatively, we could try to add some sort of cooperation between
DiagRuntimeBehavior and constant evaluation, to try to avoid having to diagnose
everyth
katzdm wrote:
@Fznamznon Hmm...my suggested change does seem to break some tests :( But to
circle back to your original question, I think it's not totally unexpected - a
broader class of variable initializers are now (correctly afaict) evaluated as
constant expressions, which could make some
katzdm wrote:
> @katzdm It seems after this a `warn_impcast_integer_precision_constant`
> warning went missing:
>
> https://godbolt.org/z/ndsPb44b4
>
> Is that expected?
We changed [this
diagnostic](https://github.com/llvm/llvm-project/blob/5ac34358181b21135851979c1c949632be5a9d32/clang/lib/
Fznamznon wrote:
@katzdm
It seems after this a `warn_impcast_integer_precision_constant` warning went
missing:
https://godbolt.org/z/ndsPb44b4
Is that expected?
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commit
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
CI failures look unrelated
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/katzdm updated
https://github.com/llvm/llvm-project/pull/89565
>From 218ffb6d5a03c96d46bfecdb4490277809f5b62e Mon Sep 17 00:00:00 2001
From: Dan Katz
Date: Tue, 16 Apr 2024 17:14:50 -0400
Subject: [PATCH 1/8] Fix bug with constexpr initialization.
---
clang/lib/Parse/ParseD
https://github.com/katzdm updated
https://github.com/llvm/llvm-project/pull/89565
>From 218ffb6d5a03c96d46bfecdb4490277809f5b62e Mon Sep 17 00:00:00 2001
From: Dan Katz
Date: Tue, 16 Apr 2024 17:14:50 -0400
Subject: [PATCH 1/8] Fix bug with constexpr initialization.
---
clang/lib/Parse/ParseD
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/89565
>From 218ffb6d5a03c96d46bfecdb4490277809f5b62e Mon Sep 17 00:00:00 2001
From: Dan Katz
Date: Tue, 16 Apr 2024 17:14:50 -0400
Subject: [PATCH 1/8] Fix bug with constexpr initialization.
---
clang/lib/Parse/Pars
https://github.com/katzdm edited https://github.com/llvm/llvm-project/pull/89565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
13 matches
Mail list logo
