Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

General: Replace GC_ALIGN macros with alignas #2972

Merged
merged 1 commit into from Sep 11, 2015

Conversation

lioncash
Copy link
Member

@lioncash lioncash commented Sep 5, 2015

Standard-supported alignment facilities ∴ out with compiler-specific.

@@ -57,7 +57,7 @@ struct tlb_entry
};

// This contains the entire state of the emulated PowerPC "Gekko" CPU.
struct GC_ALIGNED64(PowerPCState)
struct alignas(16) PowerPCState

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

Standard supported alignment -> out with compiler-specific.
@lioncash
Copy link
Member Author

lioncash commented Sep 9, 2015

Is there anything else holding this back?

@phire
Copy link
Member

phire commented Sep 9, 2015

No, everything looks good to me.

Tilka added a commit that referenced this pull request Sep 11, 2015
General: Replace GC_ALIGN macros with alignas
@Tilka Tilka merged commit c5685ba into dolphin-emu:master Sep 11, 2015
@lioncash lioncash deleted the align branch September 11, 2015 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants