<div dir="ltr">Hi Salman, it looks like patch doesn't have edit access for anyone but you. I had drafted some comments but can't hit the submit button.<div>I think you can go to <a href="https://reviews.llvm.org/differential/revision/edit/116085/">https://reviews.llvm.org/differential/revision/edit/116085/</a> and change the `Editable By` field to `All Users`</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 21, 2021 at 8:45 AM Salman Javed via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">salman-javed-nz updated this revision to Diff 395606.<br>
salman-javed-nz added a comment.<br>
<br>
Remove unnecessary `llvm::` qualification.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D116085/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D116085/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D116085" rel="noreferrer" target="_blank">https://reviews.llvm.org/D116085</a><br>
<br>
Files:<br>
  clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp<br>
  clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolint.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-LIFO.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-all-end-glob.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-all-end-specific.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-glob-end-all.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-glob-end-specific.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-global-end-specific.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-multiple-end-single.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-single-end-multiple.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-specific-end-all.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-specific-end-glob.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-specific-end-global.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-mismatched-check-names.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-mismatched-delims.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-typo-in-check-name.cpp<br>
  clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend.cpp<br>
<br>
</blockquote></div>