This is an archive of the discontinued LLVM Phabricator instance.

[Sema] Split availability processing into SemaAvailability.cpp
ClosedPublic

Authored by rnk on Jan 24 2020, 3:21 PM.

Details

Summary

Reduces compile time of SemaDeclAttr.cpp down to 28s from 50s. The new
TU does a few RecursiveASTVisitor instantiations, so it takes 30s.

Diff Detail

Event Timeline

rnk created this revision.Jan 24 2020, 3:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 24 2020, 3:21 PM
rsmith accepted this revision.Jan 24 2020, 3:31 PM
This revision is now accepted and ready to land.Jan 24 2020, 3:31 PM

Unit tests: fail. 62189 tests passed, 1 failed and 815 were skipped.

failed: libc++.std/thread/thread_mutex/thread_mutex_requirements/thread_mutex_requirements_mutex/thread_mutex_class/try_lock.pass.cpp

clang-tidy: fail. clang-tidy found 0 errors and 3 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.