llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-static-analyzer-1 Author: Balázs Kéri (balazske) <details> <summary>Changes</summary> It looks not possible to have a `sed` command with -i option that works on all platforms. The -i option is not necessary so it is removed. --- Full diff: https://github.com/llvm/llvm-project/pull/157038.diff 1 Files Affected: - (modified) clang/test/Analysis/ctu-import-type-decl-definition.c (+2-2) ``````````diff diff --git a/clang/test/Analysis/ctu-import-type-decl-definition.c b/clang/test/Analysis/ctu-import-type-decl-definition.c index 63d75ee153bed..d7965aa56e599 100644 --- a/clang/test/Analysis/ctu-import-type-decl-definition.c +++ b/clang/test/Analysis/ctu-import-type-decl-definition.c @@ -4,8 +4,8 @@ // RUN: %clang_cc1 -emit-pch -o %t/import.c.ast %t/import.c -// RUN: %clang_extdef_map -- -x c %t/import.c >> %t/externalDefMap.txt -// RUN: sed -i='' 's/$/.ast/' %t/externalDefMap.txt +// RUN: %clang_extdef_map -- -x c %t/import.c >> %t/externalDefMap_.txt +// RUN: sed 's/$/.ast/' %t/externalDefMap_.txt >> %t/externalDefMap.txt // RUN: %clang_cc1 -analyze \ // RUN: -analyzer-checker=core \ `````````` </details> https://github.com/llvm/llvm-project/pull/157038 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
