[PATCH] D142825: [llvm][NFC] Use move instead of copy

Chris Cotter via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 08:54:32 PST 2023


ccotter added a comment.

Yes, locally I get

  $ ./clang/tools/clang-format/git-clang-format HEAD~
  clang-format did not modify any files

I think (unless something has broken lately) https://buildkite.com/llvm-project/premerge-checks/builds/133226#0185fb40-6564-4910-a1ac-820fd11c0708 is running clang-format. On other PRs I've submitted over the past few weeks have caught an occasional instance where I forgot to run clang-format locally.

The message on this phabricator diff saying "clang-format not found in user’s local PATH; not linting file." is due to my local setup not having `arc diff` know how to automatically run clang-format before sending up the diff (which I need to fix).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142825/new/

https://reviews.llvm.org/D142825



More information about the llvm-commits mailing list