[PATCH] D76054: [clang-apply-replacements] No longer deduplucates replacements from the same TU

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 23 07:03:33 PDT 2020


njames93 added inline comments.


================
Comment at: clang-tools-extra/test/clang-apply-replacements/identical2.cpp:3
+// RUN: grep -Ev "// *[A-Z-]+:" %S/Inputs/identical2/identical2.cpp > %T/Inputs/identical2/identical2.cpp
+// RUN: sed "s#\$(path)#%/T/Inputs/identical2#" %S/Inputs/identical2/file1.yaml > %T/Inputs/identical2/file1.yaml
+// RUN: sed "s#\$(path)#%/T/Inputs/identical2#" %S/Inputs/identical2/file2.yaml > %T/Inputs/identical2/file2.yaml
----------------
ymandel wrote:
> Why is this `%/T` rather than `%T`? I realize it is the same in `identical.cpp`, but just want to understand what I'm reading...
It's like that in most of the checks, not sure why but wanted to keep it consistent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76054





More information about the cfe-commits mailing list