This is an archive of the discontinued LLVM Phabricator instance.

[TAPI] Update variable names to match TAPI version in unit tests. (NFC)
ClosedPublic

Authored by JDevlieghere on Jan 23 2020, 10:47 PM.

Diff Detail

Event Timeline

JDevlieghere created this revision.Jan 23 2020, 10:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 23 2020, 10:47 PM
Herald added a subscriber: dexonsmith. · View Herald Transcript
cishida accepted this revision.Jan 23 2020, 10:49 PM

LGTM, thanks!

This revision is now accepted and ready to land.Jan 23 2020, 10:49 PM

nit: probably should update [TAPI] to [llvm][TextAPI/MachO] in the commit header.

Unit tests: fail. 62150 tests passed, 5 failed and 811 were skipped.

failed: libc++.std/language_support/cmp/cmp_partialord/partialord.pass.cpp
failed: libc++.std/language_support/cmp/cmp_strongeq/cmp.strongeq.pass.cpp
failed: libc++.std/language_support/cmp/cmp_strongord/strongord.pass.cpp
failed: libc++.std/language_support/cmp/cmp_weakeq/cmp.weakeq.pass.cpp
failed: libc++.std/language_support/cmp/cmp_weakord/weakord.pass.cpp

clang-tidy: fail. clang-tidy found 0 errors and 28 warnings. 0 of them are added as review comments below (why?).

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

This revision was automatically updated to reflect the committed changes.