drt: Unifiying Priority Vias Logic

[bnmfw]

Summary

Implements the getPriorityViaDefs() function, which simply puts the logic of "The first two vias have priorities over all others in Pin Access" in a single place, instead of being independently in two places.

Users pushing PRs have built on top of the already faulty logic which lead to bad code. Putting this in one place makes things more concise and allows to in the future change this logic. The logic itself is kind of arbitrary, but I'm not changing it, just putting it in one place.

Also applies minor refactors:

• deep_search renamed to try_all_vias
• via_in_pin renamed to via_must_be_in_pin
• hasViaDef noop refactor

Type of Change

Noop refactor

Impact

Pure refactor

Verification

• I have verified that the local build succeeds (./etc/Build.sh).
• I have run the relevant tests and they pass.
• My code follows the repository's formatting guidelines.
• I have signed my commits (DCO).

Related Issues

#10937

[gemini-code-assist]

Code Review

This pull request refactors the via selection and filtering logic in FlexPA by introducing the getPriorityViaDefs helper function, simplifying the via_defs vector representation, and renaming variables for clarity. The review feedback highlights several improvement opportunities, including resolving a signed/unsigned comparison warning in frAccess.h, marking the new query function as const while avoiding potential map insertion side-effects from operator[], preventing potential underflow/negative index issues when querying layer_num - 1, and fixing a typo in the variable name via_not_int_pin.