-
Notifications
You must be signed in to change notification settings - Fork 415
Clusterer feasibility changes #1641
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 7 commits
918219f
1fdf881
ca7b577
ed6e7a8
532a143
511a994
8008b2e
54f4995
5845338
7b465fe
6fb45ff
3a606e8
4fceb1e
f00d55d
f34a1d6
e4b8257
6e59e42
ffa49a1
83cfe2b
ed0cd07
7e8f9d5
c8b58b8
f11da67
899a3da
afcb926
ee4bf86
37ef5a7
cfb812f
e2a7bfc
680cbdc
40e9534
d2d6fcb
bd217bf
8c5bc25
b807f83
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -164,6 +164,7 @@ enum e_block_pack_status { | |
BLK_PASSED, | ||
BLK_FAILED_FEASIBLE, | ||
BLK_FAILED_ROUTE, | ||
BLK_FAILED_FLOORPLANNING, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. think about whether this enum can be in a lower header file. If it's only used in the clusterer, can have a cluster_utilities header file and put this enum in there. This can be a cleanup on a separate pull request There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. in pack subdirectory - is there a cluster utilities type header file? check, could possibly put there |
||
BLK_STATUS_UNDEFINED | ||
}; | ||
|
||
|
Uh oh!
There was an error while loading. Please reload this page.