This is an archive of the discontinued LLVM Phabricator instance.

Adding Pass Dependencies for CodeGenPrepare
ClosedPublic

Authored by AndrewLitteken on Jul 22 2020, 10:25 AM.

Details

Summary

When adding a pass to the pipeline, there would be differences between in the generated code when the pass performed no operations, and when the pass was not present.

This pass adds dependencies for

  • TargetTransformInfoWrapperPass
  • TargetPassConfig
  • LoopInfoWrapperPass
  • TargetLibraryInfoWrapperPass

which are used but did not were not recorded using the INITIALIZE_PASS_DEPENDENCY macro to fix this inconsistency.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJul 22 2020, 10:25 AM

LGTM with one minor comment

llvm/lib/CodeGen/CodeGenPrepare.cpp
446

Can you alphabetize these?

Alphabetizing pass dependencies

AndrewLitteken marked an inline comment as done.Jul 22 2020, 11:02 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 22 2020, 12:03 PM
This revision was automatically updated to reflect the committed changes.