Page MenuHomePhabricator

[CodeGenPrepare] Make TargetPassConfig required

Authored by MaskRay on Jan 30 2020, 4:52 PM.



The code paths in the absence of TargetMachine, TargetLowering or
TargetRegisterInfo are poorly tested. As rL285987 said, requiring
TargetPassConfig allows us to delete many (untested) checks littered

Diff Detail

Event Timeline

MaskRay created this revision.Jan 30 2020, 4:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 30 2020, 4:52 PM

Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped.

failed: libc++.std/containers/sequences/array/array_creation/

clang-tidy: pass.

clang-format: pass.

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.

arsenm accepted this revision.Jan 31 2020, 7:44 AM
arsenm added inline comments.

This should also be a long dead check, but that's a separate change

This revision is now accepted and ready to land.Jan 31 2020, 7:44 AM
This revision was automatically updated to reflect the committed changes.
MaskRay marked 2 inline comments as done.Feb 2 2020, 11:53 AM
MaskRay added inline comments.