This is related to D94982. We want to call these APIs from the Analysis
component, so we can't leave them under Transforms.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/Analysis/ImportedFunctionsInliningStatistics.cpp | ||
---|---|---|
189 | NOTE: this was the linter having initiative - I'd leave the change in, albeit unrelated. |
In your commit the message does not include Reviewed by:. Many people agree that both Reviewed by: & Differential Revision: should be present. The Reviewed by: list indicates people who acknowledged the patch. (The Reviewers: list does not necessarily mean all the people on the list have acknowledged the patch so Reviewers: is mostly useless.)
arc amend can fetch the Phabricator summary and amend the local description.
You can install llvm/utils/git/pre-push.py to prevent accidental Summary:, Reviewers:, Subscribers: and Tags: in the presence of Differential Revision:.
FWIW, my take on it would be that including "Reviewed by:" is not harmful/may be useful (as opposed to the other fields (except Differential Revision), which I think actively add noise to the commit messages), but I wouldn't say it's required/don't think it rises to the level of needing to be fixed/corrected.