Skip to content

Comments

grammar/changelog for asm attributes#667

Merged
DmitryOlshansky merged 1 commit intodlang:masterfrom
MartinNowak:asmAttribute
Nov 27, 2014
Merged

grammar/changelog for asm attributes#667
DmitryOlshansky merged 1 commit intodlang:masterfrom
MartinNowak:asmAttribute

Conversation

@MartinNowak
Copy link
Member

No description provided.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, but then why allow the attributes at all?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So that the compiler knows how your asm code behaves, I added a reasoning to the changelog.

@MartinNowak
Copy link
Member Author

Would be nice to get this merged.
Also to start with the changelog for 2.067.

@DmitryOlshansky
Copy link
Member

Can't auto-merge this on github. Rebase?

@MartinNowak
Copy link
Member Author

Updated, I also change the wording in the changelog to emphasize why one can add attributes.

@MartinNowak
Copy link
Member Author

Ping

DmitryOlshansky added a commit that referenced this pull request Nov 27, 2014
grammar/changelog for asm attributes
@DmitryOlshansky DmitryOlshansky merged commit 3616975 into dlang:master Nov 27, 2014
@MartinNowak MartinNowak deleted the asmAttribute branch November 28, 2014 01:19
@MartinNowak
Copy link
Member Author

Thanks

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