Skip to content

Conversation

komh
Copy link
Contributor

@komh komh commented Nov 27, 2024

On any other OMF assemblers such as MASM, TASM and ALP, a group is cumulative.

@komh komh force-pushed the cumulative_group branch 3 times, most recently from 10e501f to 17d62ba Compare November 29, 2024 10:50
On any other OMF assemblers such as MASM, TASM and ALP, a group is
cumulative.

Signed-off-by: KO Myung-Hun <[email protected]>
@hpax
Copy link
Member

hpax commented Oct 4, 2025

To be frank, I don't even really understand what "groups are cumulative" mean. If you could put something in changes.src (and, if necessary, in outfmt.src) that would be really helpful.

@ecm-pushbx
Copy link
Contributor

@hpax I think they mean that multiple group directives naming the same group can be used, and all section names listed in any of them are accumulated into the same group. I came across this during some work porting source texts to NASM. NASM currently rejects multiple group directives naming the same group.

@komh
Copy link
Contributor Author

komh commented Oct 5, 2025

For example,

group DGROUP .data
group DGROUP .bss
group DGROUP .data

should put .data, .bss in DGROUP without any problems. Currently, NASM does not allow this. I'll add this explanation to changes.src and outfmt.src.

@komh komh force-pushed the cumulative_group branch from 17d62ba to f558be8 Compare October 5, 2025 08:23
@komh
Copy link
Contributor Author

komh commented Oct 5, 2025

I've documented for cumulative GROUP directive.

@hpax
Copy link
Member

hpax commented Oct 6, 2025

Merged.

@hpax hpax closed this Oct 6, 2025
@komh komh deleted the cumulative_group branch October 8, 2025 13:56
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.

3 participants