[PATCH] D96816: [ObjC] Encode pointers to C++ classes as "^v" if the encoded string would otherwise include template specialization types

Akira Hatanaka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 18 11:06:00 PST 2021


ahatanak added a comment.

In D96816#2572218 <https://reviews.llvm.org/D96816#2572218>, @thakis wrote:

> Very cool change, I wanted something like this for a long time :)
>
> It seems to break Index/preamble-reparse-changed-module.m  on some platforms:
> http://45.33.8.238/linux/39727/step_7.txt
> http://45.33.8.238/macm1/3788/step_6.txt
>
> Does the failure make sense to you?

I don't know why the test is failing. It doesn't fail on my machine.

This is also broken:  http://green.lab.llvm.org/green/job/lldb-cmake/29029/

I'm not sure if it's related to the one you saw.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96816/new/

https://reviews.llvm.org/D96816



More information about the cfe-commits mailing list