This is an archive of the discontinued LLVM Phabricator instance.

Differential D127217

[include-cleaner] Fix build error in unit test
ClosedPublic

Authored by ckandeler on Jun 7 2022, 7:30 AM.

Download Raw Diff

Details

Reviewers

sammccall
nridge

Commits

rGd3520171844a: [include-cleaner] Fix build error in unit test

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

ckandeler created this revision.Jun 7 2022, 7:30 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2022, 7:30 AM

Herald added a subscriber: mgorny. · View Herald Transcript

ckandeler requested review of this revision.Jun 7 2022, 7:30 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2022, 7:30 AM

Herald added a subscriber: cfe-commits. · View Herald Transcript

Harbormaster completed remote builds in B168300: Diff 434811.Jun 7 2022, 7:49 AM

ckandeler added a reviewer: nridge.Jun 8 2022, 7:54 AM

Thanks!

This revision is now accepted and ready to land.Jun 8 2022, 12:43 PM

Can you please merge it? I don't have the rights.

Closed by commit rGd3520171844a: [include-cleaner] Fix build error in unit test (authored by ckandeler, committed by nridge). · Explain WhyJun 9 2022, 12:38 AM

This revision was automatically updated to reflect the committed changes.

nridge added a commit: rGd3520171844a: [include-cleaner] Fix build error in unit test.

Revision Contents

Path

Size

clang-tools-extra/

include-cleaner/

unittests/

1 line

Diff 435438

clang-tools-extra/include-cleaner/unittests/CMakeLists.txt

	Show All 17 Lines		clang_target_link_libraries(ClangIncludeCleanerTests
	clangBasic			clangBasic
	clangFrontend			clangFrontend
	)			)

	target_link_libraries(ClangIncludeCleanerTests			target_link_libraries(ClangIncludeCleanerTests
	PRIVATE			PRIVATE
	clangIncludeCleaner			clangIncludeCleaner
	clangTesting			clangTesting
				LLVMTestingSupport
	)			)