tree 6f075eefaea9f22f0a81f64bb1e7d399a42a04f3
parent ebd16fd5dd23b2809705e3e6fcaf549f2ac0b8ae
author Kalen Brunham <kalen.brunham@intel.com> 1765504699 +0000
committer Kalen Brunham <kalen.brunham@intel.com> 1765504699 +0000

Fix formatter merging compiler directives onto single line

Fixes #2073

The formatter was incorrectly merging consecutive compiler directives
(like `timescale and `default_nettype) onto a single line instead of
keeping them on separate lines.

Root cause: IsPreprocessorKeyword() only recognized preprocessor control
flow directives (PP_* tokens) but not compiler directives (DR_* tokens),
and the tree unwrapper wasn't creating separate partitions for these
directives.

Changes:
- Extended IsPreprocessorKeyword() to include all DR_* compiler directive
  tokens (timescale, default_nettype, resetall, celldefine, etc.)
- Added kTimescaleDirective and kTopLevelDirective nodes to unindented
  unwrapped line handling in tree unwrapper
- Added leaf-level handling for bare DR_* tokens to ensure they start
  new partitions when at top level
- Added comprehensive regression tests for multiple directive combinations

All tests pass. Directives now correctly remain on separate lines when
at top level, while respecting indentation when inside module ports or
other nested contexts.