[PATCH] D58507: [MIPS] [microMIPS] Fix for filling delay slots for PseudoIndirectBranch_MM
Simon Dardis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 21 13:48:56 PST 2019
sdardis added a subscriber: llvm-commits.
sdardis added a comment.
Sorry I didn't spot this earlier, but in future please ensure 'llvm-commits' is one of the subscribers when creating a review request for LLVM. If you add it after creating a review request, manually add it and write something in the comments field to trigger Phabricator into sending an email or abandon the review request and re-open it with the relevant -commits list as an initial subscriber.
Posting review requests without the relevant -commits list means that only the subscribers added, subscribers added through Herald rules and initial reviewers will see the request. It is policy that patches are emailed to the relevant list for review[1]. Submitting patches through Phabricator is fine, provided the relevant -commits list is in the subscribers.
Thanks.
[1] http://www.llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58507/new/
https://reviews.llvm.org/D58507
More information about the llvm-commits
mailing list