steakhal wrote: > The change itself looks good to me, but I wonder what is the reason that we > only want to do this canonicalization when we access fields?
It took me a while to unwrap your question. Actually, think of it as an invariant. We should never have reference typed value regions wrapped by subregions. This is checked/enforced by how we create subregions (see the asserts). The second is, how can we end up with such subregions. I think the only way is by field regions. If we had something that I missed, the assert would catch it. Do you have something in mind? https://github.com/llvm/llvm-project/pull/118096 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
