Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/117...@github.com>


erichkeane wrote:

> Meh, saving a `ConstantExpr` in the `FieldDecl` doesn't work properly, since 
> we need to support having arbitrary exprs in case the bitwidth is dependent.

The idea is that you'd store an `Expr` like we do elsewhere, and then upon 
instantiating it/evaluating it, we'd replace it with a `ConstantExpr`.  So you 
don't need separate storage, just replace the current thing.

The instantiation of a `ConstantExpr` is to just return it, so you don't end up 
losing anything as a side effect.

https://github.com/llvm/llvm-project/pull/117732
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to