Skip to content

Commit

Permalink
[XLS] Only run LUT conversion pass once
Browse files Browse the repository at this point in the history
This pass is about to get a bit too expensive to run in every fixed-point simplification pass.

PiperOrigin-RevId: 700533971
  • Loading branch information
ericastor authored and copybara-github committed Nov 27, 2024
1 parent 693ecc2 commit 62e2a7e
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions xls/passes/optimization_pass_pipeline.cc
Original file line number Diff line number Diff line change
Expand Up @@ -107,8 +107,6 @@ void AddSimplificationPasses(OptimizationCompoundPass& pass) {
pass.Add<DeadCodeEliminationPass>();
pass.Add<DataflowSimplificationPass>();
pass.Add<DeadCodeEliminationPass>();
pass.Add<LutConversionPass>();
pass.Add<DeadCodeEliminationPass>();
pass.Add<ConditionalSpecializationPass>(/*use_bdd=*/false);
pass.Add<DeadCodeEliminationPass>();
pass.Add<ReassociationPass>();
Expand Down Expand Up @@ -258,6 +256,9 @@ class PostInliningOptPassGroup : public OptimizationCompoundPass {
Add<SelectLiftingPass>();
Add<DeadCodeEliminationPass>();

Add<LutConversionPass>();
Add<DeadCodeEliminationPass>();

Add<ConditionalSpecializationPass>(/*use_bdd=*/true);
Add<DeadCodeEliminationPass>();

Expand Down

0 comments on commit 62e2a7e

Please sign in to comment.