[PATCH] D91900: OpaquePtr: Make byval/sret types mandatory

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 19:59:53 PST 2020


dblaikie added a comment.

In D91900#2409296 <https://reviews.llvm.org/D91900#2409296>, @arsenm wrote:

> 41083267a9d7a006cf5821344480650becd03c3c <https://reviews.llvm.org/rG41083267a9d7a006cf5821344480650becd03c3c>

FWIW, if you include the `Differential Revision: https:\\reviews.llvm.org\DXXXX` in the commit description, it'll autoclose the review so you don't have to & record the revision here, show any trailing diffs between the review and the commit, and the link comes in handy when reading the revision history to quickly/easily jump to the review for more context.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91900/new/

https://reviews.llvm.org/D91900



More information about the llvm-commits mailing list