[llvm-dev] Does LLVM still keep the SVN review history?

David Blaikie via llvm-dev llvm-dev at lists.llvm.org
Wed Jul 29 10:33:40 PDT 2020


Not all patches are reviewed in Phabricator - the LLVM mailing list is
still the source of truth for review discussions. Here's the review
thread for that commit:
http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20150105/251381.html
- back in the day, before Phabricator was used at all (& this is still
a valid, though less-used strategy), email threads on llvm-commits
would review the patch, then once the patch was committed a follow-up
email to the original thread would mention the revision that was
committed with the results of the review. I don't know of an effective
way to search the mailing list archive short of having subscribed to
the mailing list and kept all the email... but if you happen to have,
when you search for "r226592" you'll find the original commit email
and the original review thread.

rdar://*** is an Apple-internal bug tracker link. It's not useful to
anyone outside of Apple. (there's some debate about whether it's
acceptable to put these kinds of internal-only information in public
communications/bugs/commits - but in any case, even if they're
included, all the publicly necessary context should be rendered in the
public communications - commit messages, bug descriptions, etc)

On Wed, Jul 29, 2020 at 10:24 AM Stephan Z via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
>
> Hi,
>
> For example,  https://reviews.llvm.org/rGdab999d54f39af3ebb6a23e850dbc9742016fac8 looks like an SVN commit, and I did not find its review history from Phabricator.
>
> It also mentions rdar://19040716. Is this link still valid to access?
>
> Thank you,
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev


More information about the llvm-dev mailing list