fixes #23499; don't skip addr when constructing bracketExpr

[ringabout]

fixes #23499

In the 8990626 the effect of skipAddr changed to skip nkAddr and nkHiddenAddr. Some old code was not adapted. In the #23477, the magic addr function was handled in the semantic analysis phase, which causes it be skipped incorrectly

[ringabout]

/home/runner/.nimble/pkgs2/arraymancer-0.7.28-d4a45ada1c7a6abebe60bcdd5ee2d7c4680799a4/arraymancer/tensor/private/p_accessors.nim(229, 50) Error: undeclared identifier: 't2data`gensym212'; if declared in a template, this identifier may be inconsistently marked inject or gensym

Unrelated CI failures

[github-actions]

Thanks for your hard work on this PR!
The lines below are statistics of the Nim compiler built from 549ef24

Hint: mm: orc; opt: speed; options: -d:release
178475 lines; 8.271s; 752.285MiB peakmem

[Vindaar]

/home/runner/.nimble/pkgs2/arraymancer-0.7.28-d4a45ada1c7a6abebe60bcdd5ee2d7c4680799a4/arraymancer/tensor/private/p_accessors.nim(229, 50) Error: undeclared identifier: 't2data`gensym212'; if declared in a template, this identifier may be inconsistently marked inject or gensym

Unrelated CI failures

I just tagged a new arraymancer version. That should fix the CI failures here and elsewhere. Relevant PR that fixes it on the arraymancer side should be @metagn's mratsim/Arraymancer#642.