<div dir="ltr">Hi Raphael,<div><br></div><div>I have a very similar test for a tool that integrates with lldb which failed without the patch.</div><div>I thought the test in the patch would behave the same which is apparently not the case.</div><div>Thanks for pointing that out. I will need to take another look and will get back to you once I figure out what's wrong.</div><div>Sorry for the inconvenience.</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">kind regards,<div>Lasse</div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 2, 2021 at 1:15 PM Raphael “Teemperor” Isemann <<a href="mailto:teemperor@gmail.com">teemperor@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;">Hi Lasse,<div><br></div><div>the test from the patch passes for me even without your non-test changes. Not sure if you attached the wrong diff or it needs to be applied on a specific commit that is not ToT? Can you maybe try pushing your code to some git repo?</div><div><br></div><div>Your change to TypeSystemClang (which I assume removes the canonicalization of <span style="color:rgb(9,46,100)">parent_qual_type) is from what I can see not actually changing the result value of `GetChildCompilerTypeAtIndex`. It looks like the return value for pointer types is computed independently from `parent_qual_type` without any canonicalization.</span></div><div><span style="color:rgb(9,46,100)"><br></span></div><div><span style="color:rgb(9,46,100)">Cheers,</span></div><div><span style="color:rgb(9,46,100)">- Raphael</span></div><div><div><br><blockquote type="cite"><div>On 2 Jun 2021, at 11:39, Lasse Folger via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:</div><br><div><span id="gmail-m_1316216663779864151cid:f_kpf9kaor0"><0001-lldb-prevent-canonization-of-type-when-dereferencing.patch></span></div></blockquote></div><br></div></div></blockquote></div>