[llvm] [Support] Don't use StringRef::equals in Path.inc (PR #98839)
Rainer Orth via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 16 04:34:14 PDT 2024
rorth wrote:
Thanks for the clarification. In the old days, before LLVM switched to pull requests, I'd just have committed the patch, especially in a case like this with an obvious patch to unbreak the build. However, I wasn't sure if this was appropriate in the new world, so meant to err on the side of caution while still restoring the build quickly.
Besides, I meant to alert Kazu, wondering if he'd missed the buildbot failure notification in [[ADT] Remove StringRef::equals](https://github.com/llvm/llvm-project/pull/98735) which wouldn't work with a simple commit.
Another advantage of the pull request route is that you get a verification of the description's markup.
> If you only wanted to create the pull request to run automated presubmit checks, please create the PR with the skip-precommit-approval to indicate that)
That would be `git commit --no-verify`?
https://github.com/llvm/llvm-project/pull/98839
More information about the llvm-commits
mailing list