Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
This wasn't trapped by my filters when it was first posted - sorry about that. In future if it's a mips related patch you can add me directly as a reviewer when posting the patch. Changing the subscribers doesn't nag the list or the newly added person. Also, please upload patches with context - http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface
From my reading of the GCC sources with OpenBSD's patches and r282184, LGTM.
Good to know about the reviewer.
I am not familiar with phabricator and was just trying something with the addition of the subscriber.
I will keep the amount of context in mind for future diffs.
Thanks.
FYI, you should close this revision manually and cite the commit (rL297098). Also please cite the review and all the reviewers when you commit. E.g. the commit message for that should be:
[mips] Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64
Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64.
Reviewers: sdardis
Differential Revision: https://reviews.llvm.org/D30465
Doing all this helps with auditing commits and future maintenance. Phab will pick up on commit messages in that form and close the review automatically. The arcanist tool will automate adding the necessary lines to the commit message.