Issue 24790 - -vcg-ast ICE on lowered assign exp
Summary: -vcg-ast ICE on lowered assign exp
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: dmd (show other issues)
Version: D2
Hardware: All All
: P3 normal
Assignee: No Owner
URL:
Keywords: ice, pull
Depends on:
Blocks:
 
Reported: 2024-10-03 09:56 UTC by Dennis
Modified: 2024-11-16 23:27 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Dennis 2024-10-03 09:56:32 UTC 
```D
struct S { int[] payload; }
long f(S s) => s.payload.length += 3;
```

Compiling with -vcg-ast trips an assert with "precedence not defined for token '='", because there's no PREC entry for EXP.loweredAssignExp.
Comment 1 Dlang Bot 2024-10-03 10:01:40 UTC 
@dkorpel created dlang/dmd pull request #16914 "Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp" fixing this issue:

- Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp

https://github.com/dlang/dmd/pull/16914
Comment 2 Dlang Bot 2024-10-03 11:46:46 UTC 
dlang/dmd pull request #16914 "Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp" was merged into stable:

- 1f0135d6459e38dc49393590bc83fa2a8ed300e0 by Dennis Korpel:
  Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp

https://github.com/dlang/dmd/pull/16914
Comment 3 Dlang Bot 2024-10-07 09:00:29 UTC 
dlang/dmd pull request #16967 "Rebase stable6" was merged into master:

- ca699f1849ac16296243e6dd048330b744021b39 by Dennis:
  Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp (#16914)
  
  Co-authored-by: Dennis Korpel <dennis@sarc.nl>

https://github.com/dlang/dmd/pull/16967
Comment 4 Dlang Bot 2024-11-16 23:27:14 UTC 
dlang/dmd pull request #17069 "Merge stable" was merged into master:

- f420f988ab1f53e604851c73e9f883dab4328f65 by Dennis:
  Fix bugzilla 24790 - -vcg-ast ICE on lowered assign exp (#16914)
  
  Co-authored-by: Dennis Korpel <dennis@sarc.nl>

https://github.com/dlang/dmd/pull/17069