This is an archive of the discontinued LLVM Phabricator instance.

[mips] Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64
ClosedPublic

Authored by brad on Feb 28 2017, 9:06 AM.

Details

Reviewers
sdardis
Summary

Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64

Diff Detail

Repository
rL LLVM

Event Timeline

brad created this revision.Feb 28 2017, 9:06 AM
brad added a subscriber: sdardis.Feb 28 2017, 11:19 AM
sdardis accepted this revision.Mar 6 2017, 6:56 AM

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.

This revision is now accepted and ready to land.Mar 6 2017, 6:56 AM
brad added a comment.Mar 6 2017, 3:55 PM

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.

Do you need me to commit this for you?

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.

sdardis closed this revision.Aug 11 2017, 3:17 AM

Closing this, rL297098.