[Chapel Merge] fix(chpldoc): address failed smoketest on mac-over

Notifications Commits
1

Branch: refs/heads/main
Revision: cb934ad
Author: arezaii
Link: Unavailable
Log Message:

Merge pull request #20598 from arezaii/fix-chpldoc-macbuild

fix(chpldoc): address failed smoketest on mac-overflow

This PR fixes an error reported by chapelmac smoketests.

The compiler did not like
const auto& ast : builderResult->topLevelExpressions())
and complained about the iterator being copied on every loop.
It suggested replacing the statement with
const chpl::uast::AstNode* ast : builderResult->topLevelExpressions())
which is the change that I implemented.

TESTING:

  • test/chpldocs all pass
  • fixes failure mode on chapelmac machine

reviewed by @benharsh - thank you!

Signed-off-by: arezaii ahmad.rezaii@hpe.com

Modified Files:
M compiler/dyno/tools/chpldoc/chpldoc.cpp

Compare: https://github.com/chapel-lang/chapel/compare/83cd81dc5d0a...cb934adec463