Skip to content

Conversation

@jabraham17
Copy link
Member

@jabraham17 jabraham17 commented Jan 5, 2026

Fixes an issue where we were erroneously throwing an internal error when calling uniquifyName when --debug-safe-optimizations-only was passed.

I narrowed this down to being caused by --no-copy-propagation --no-dead-code-elimination leaving behind multiple extern declarations with the same name in a single function. This failure is caused by the compiler leaving extra temporaries behind, but the bug is actually just #9299, so I just fixed #9299.

I also found a failure when using remove-empty-records by itself, which I fixed in this PR.

Resolves #28165
Resolves #9299

  • paratest with/without gasnet
  • confirmed small reproducer passes with this change
  • confirmed that original reproducer in Arkouda with COMM=gasnet and --debug-safe-optimizations-only now compiles
    • this is only partially true. Arkouda now hits a different compiler error which will be resolved in a later PR

[Reviewed by @benharsh]

Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
@jabraham17 jabraham17 requested a review from benharsh January 8, 2026 00:51
Signed-off-by: Jade Abraham <[email protected]>
@jabraham17 jabraham17 merged commit b4660d8 into chapel-lang:main Jan 8, 2026
10 checks passed
@jabraham17 jabraham17 deleted the fix-uniq branch January 8, 2026 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: --debug-safe-optimizations-only may cause crash due to uniquifyName failing uniquifyName in presence of extern symbols

2 participants