miniblog.

← Back to all posts
Feb 27, 2022 at 06:11
Phabricator does a great job displaying diffs. It has subword highlighting and puts the previous section (based on indentation) in the header, shown as `namespace llvm {`. The moved line highlighting is a really nice touch too. (screenshots of https://reviews.llvm.org/D71126)
PhotoPhoto
⚙ D71126 Adding PoisonValue for representing poison value explicitly in IR
X / ?

Related Posts

Jul 25, 2024 at 16:25
I'm increasingly doubtful that commit messages should be entirely immutable. I sometimes find myself editing or commenting on merged pull requests, to help future readers. Phabricator appended a URL to commit messages, which helped. In principle PR merging could do the same thing.
Jan 10, 2022 at 07:52
Phabricator has 'moved line' detection in its diffs, and a comprehensible UI for it! Source:
Photo
⚙ D116871 [llvm-reduce] optionally keep running delta passes until they stop making the IR smaller
X / ?
Aug 10, 2019 at 23:54
One Phabricator feature I really miss in GitHub is persistently associating commits with PRs. Phabricator commit messages have a specific syntax to show what change they're associated with. You can amend the commit without losing that metadata.