MTC added a comment.
Herald added a subscriber: martong.
Hi @NoQ, sorry to bother you. This patch has been on hold for a very long time.
It should not have been fixed yet. What's next? Abandon the patch, or improve
it yourself?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://review
NoQ added a comment.
Ah, the good old "stringly typed" APIs.
Would you like to write a LIT test for the fix, say by `FileCheck`ing the Plist
output? (i could write one for you if it's not your cup of tea)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67149/new/
MTC added a comment.
Good catch, the fix is reasonable to me. However, I'm no longer wok on the
analyzer now, maybe you can add @NoQ Or some other active reviewers to review
the code.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67149/new/
https://reviews.llv
rtmpdavid created this revision.
rtmpdavid added a reviewer: MTC.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
There is a mixup in this specific one, name and category are in incorrect order
Repository:
rC Clang
https://reviews.llvm.org/D67149
Files:
clang/lib/Sta