dgoldman marked 3 inline comments as done.
dgoldman added inline comments.

================
Comment at: clang-tools-extra/clangd/unittests/FindTargetTests.cpp:1048
         )cpp",
-           "0: targets = {global}\n"
-           "1: targets = {param}\n"
----------------
sammccall wrote:
> there are a ton of whitespace changes in this patch, because the version at 
> HEAD isn't clang-format clean :-(
> 
> Feel free to format it first in a separate commit, but I'd prefer not to 
> reformat all the tests mixed in with other changes.
Will do if it hasn't been done already


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98984

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to