[PATCH] D110068: [Clang][AST] Resolve FIXME: Remove ObjCObjectPointer from isSpecifierType

Alf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 25 06:08:19 PDT 2021


gAlfonso-bit added a comment.

In D110068#3013636 <https://reviews.llvm.org/D110068#3013636>, @dexonsmith wrote:

> I'm a bit hesitant to trust lack of tests as proof that old FIXMEs can be resolved safely. Can you add some background information about why this was originally a specifier, and why it's safe to fix it now? (If you don't know, I suggest looking through git-blame to figure it out (unless @ahatanak already knows?).)

I did and it was originally put there when the function was made. Not because of a bug.


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

https://reviews.llvm.org/D110068



More information about the cfe-commits mailing list