hvdijk wrote:

Both agreed with @nikic that this PR is a good idea and with @efriedma-quic 
that this does not solve the problem in general. I don't think the latter 
should stop this from being merged but it would be nice to find something that 
addresses all cases.

> Another fix may be to generate `DW_OP_LLVM_fragment` expressions when 
> removing the `or`

Maybe a bad suggestion, I have not looked into the implementation, but at the 
point where the `i128` gets split into two `i64`s, could we already update the 
`#dbg_value` there to use fragments? At that point we have all the required 
information, and it sounds like it would avoid the problem since there is then 
no longer any larger-than-`i64` debug info to cause `poison`?

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

Reply via email to