OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

drt: validate ap via

Open bnmfw opened this issue 8 months ago • 7 comments

Supports #7153

ISPD and Secure CI running.

Moves some of the code from filterViaAccess to the new validateAPForVia. The new function takes an Access Point and a Via and validate whether or note the via can be used to access it.

Creates a similar validateAPForPlanarAccess which is moved 1 call back to separate Planar access verification from via access verification.

filterMultipleAPAccesses was renamed to filterMultipleViaAccess since it only handles vias now.

Moves the generation of layer polys some calls back so it is calculated fewer times.

Simplification the obtuse deep search logic to just waiting for 2 valid vias.

This change made some of the unit tests use different vias on different tests, I'm not 100% sure why.

bnmfw avatar Apr 23 '25 19:04 bnmfw

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Apr 23 '25 19:04 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 05 '25 20:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 08 '25 16:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 12 '25 22:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 19 '25 14:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 19 '25 15:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 19 '25 15:05 github-actions[bot]