- User Since
- Sep 7 2015, 10:13 PM (210 w, 4 d)
Thu, Sep 19
Wed, Sep 18
Fixed a mistake where the diagnostic would print the wrong parameter type.
Here's a new version of the patch that uses iterator ranges instead of ArrayRef, to avoid adding new uses of CallExpr::getArgs() in case it has to be removed in the future due to the strict aliasing issue.
Tue, Sep 17
Ugh, it looks like getArgs() is a massive strict aliasing violation. And it's used in enough different places in Clang that fixing the violation may require extensive refactoring... I've reported the issue as https://bugs.llvm.org/show_bug.cgi?id=43344, but I don't really want to fix it :)
Mon, Sep 16
Sounds good; I'll split this into a few separate patches.
Ping. I'm not qualified to review this myself but I'd like to see the bug fixed. :)
Tue, Aug 27
Mon, Aug 26
Heh, I guess I'll request commit access, although I'm not sure if I have enough of a 'track record of submitting high quality patches'. But for now, can you commit this? Thanks :)
Addressed review comments.
Aug 21 2019
Changes since last version:
Aug 18 2019
Aug 11 2019
Oops, I forgot to re-run git diff after fixing the patch.
May 13 2019
Thanks for the feedback, updated.
May 8 2019
Oops, forgot to mention:
May 1 2019
Apr 16 2019
Thanks for the review. I don't have commit access.
Mar 25 2019
Feb 25 2019
Feb 12 2019
Oh, forgot to mention in the initial submission: I don't have commit access so I'll need someone else to commit this.
Feb 11 2019
Jul 2 2018
Fix disassembling ARM instructions as big-endian
Jul 1 2018
Mar 11 2018
Jan 21 2018
Sorry for the delay; here's an updated diff with a test. (The test didn't seem to clearly belong in any existing .s file I saw, so I just made a new one.)
Dec 4 2017
Jan 13 2016
Ping, and adding potential reviewers like I was supposed to do in the first place.
Jan 5 2016
Dec 9 2015
Nov 10 2015
I don't have SVN access. If you want to commit it, feel free to remove that comment, or else I'll update the diff tomorrow.
Addressed comments. Thanks, rsmith!
Nov 5 2015
(Ping? I verified yesterday that the patch still applies and passes regression tests.)
Oct 21 2015
One more ping. As far as I know, everything has been addressed.
Oct 13 2015
Oct 12 2015
Fixed raised issues.
Sep 26 2015
Sep 14 2015
- Changed to an explicit switch, and changed ContainsPlaceholderType logic.