[llvm] [Support] Don't use StringRef::equals in Path.inc (PR #98839)

[David Blaikie via llvm-commits]

dwblaikie 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.

Yep yep, just mentioning that it is the same norms for now, pre or post switch to github PRs. Maybe one day we'll move to PRs only, but even then, hopefully something like a build fix will be relatively low-cost, maybe not require human review, etc.

> 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`?

I don't think it's wired up to anything like that - something you can manually select in the web UI, maybe there's a way to add labels from the command line, but I'm not sure how.

https://github.com/llvm/llvm-project/pull/98839