This is an archive of the discontinued LLVM Phabricator instance.

Simplifying lit script for remove-cstr-calls
ClosedPublic

Authored by revane on Oct 31 2012, 8:28 AM.

Download Raw Diff

Details

Reviewers

klimek
chandlerc

Summary

Following the example from loop-convert tests, simplifying the remove-cstr-calls lit script.

Diff Detail

Event Timeline

From: Matthieu Monrocq [mailto:matthieu.monrocq@gmail.com]
Sent: Wednesday, October 31, 2012 2:17 PM
To: reviews+D92+public+254f3887f2e36b94@llvm-reviews.chandlerc.com
Cc: chandlerc@gmail.com; Vane, Edwin; Clang Dev List
Subject: Re: [cfe-commits] [PATCH] Simplifying lit script for remove-cstr-calls

Looks good. Thanks!

Submitted by Manuel Klimek as r167390.

Revision Contents

Path

Size

test/

remove-cstr-calls/

basic.cpp

11 lines

Diff 239

test/remove-cstr-calls/basic.cpp

	// RUN: rm -rf %t			// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp
	// RUN: mkdir %t			// RUN: remove-cstr-calls . %t.cpp --
	// RUN: echo "[{\"directory\":\".\",\"command\":\"clang++ -c %t/test.cpp\",\"file\":\"%t/test.cpp\"}]" \| sed -e 's/\\/\//g' > %t/compile_commands.json			// RUN: FileCheck -input-file=%t.cpp %s
	// RUN: cp "%s" "%t/test.cpp"
	// RUN: remove-cstr-calls "%t" "%t/test.cpp"
	// RUN: cat "%t/test.cpp" \| FileCheck %s
	// REQUIRES: shell			// REQUIRES: shell
	// FIXME: implement a mode for refactoring tools that takes input from stdin
	// and writes output to stdout for easier testing of tools.

	namespace std {			namespace std {
	template<typename T> class allocator {};			template<typename T> class allocator {};