This is an archive of the discontinued LLVM Phabricator instance.

Differential D56216

Rename TapiTests to TextAPITests
ClosedPublic

Authored by thakis on Jan 2 2019, 12:40 PM.

Details

Reviewers
amontanez
ributzka
Commits
rG4f13d041ca99: Rename TapiTests to TextAPITests
rL350339: Rename TapiTests to TextAPITests
Summary

This makes the target name consistent with how all the other unit tests are named.

Diff Detail

Event Timeline

thakis created this revision.Jan 2 2019, 12:40 PM
Herald added a subscriber: mgorny. · View Herald TranscriptJan 2 2019, 12:40 PM
thakis added a child revision: D56217: [gn build] Add even more build files for LLVM unittests.Jan 2 2019, 12:55 PM
amontanez accepted this revision.Jan 3 2019, 10:00 AM

LGTM.

This revision is now accepted and ready to land.Jan 3 2019, 10:00 AM
Closed by commit rL350339: Rename TapiTests to TextAPITests (authored by nico). · Explain WhyJan 3 2019, 10:28 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
unittests/
TextAPI/
4 lines

Diff 179925

unittests/TextAPI/CMakeLists.txt

set(LLVM_LINK_COMPONENTS set(LLVM_LINK_COMPONENTS
TextAPI TextAPI
) )
add_llvm_unittest(TapiTests add_llvm_unittest(TextAPITests
ELFYAMLTest.cpp ELFYAMLTest.cpp
) )
target_link_libraries(TapiTests PRIVATE LLVMTestingSupport) target_link_libraries(TextAPITests PRIVATE LLVMTestingSupport)