John Wright

Other hierarchical checks: - [ ] (related to #401) if using a hierarchical flow, top-level bump placements should be an error- we should force them to be placed inside the...

See #281

To be more descriptive, there's no good place to put Cadence-wide common code, for example.

This is likely related to #401, if not the same

Pseudocode to fix this: ``` if get_hierarchical_mode == "hierarchical": top_module_settings = get_hierarchical_settings(top_hierarchical_module) bumps_mode = top_module_settings["vlsi.inputs.bumps_mode"] bumps = top_module_settings["vlsi.inputs.bumps"] else: bumps_mode = get_setting("vlsi.inputs.bumps_mode") bumps = get_setting("vlsi.inputs.bumps") ```

While we do need to fix this, I'm lowering the priority because we aren't going to be doing a chip-on-board tapeout for a while

After an extremely quick glance this looks good, but I'll dive a little deeper tomorrow.

Do you have to re-run synthesis?

I understand that, this is for development. If I change something that would change it, it would be nice to not have to re-run synthesis. I think this is more...

We also may want to have a (per-library or per-project?) option to just tell hammer to pick another corner for the library if the one you want doesn't exist.