From 93db8b7ca7b7bee6565190fa0315eda2fad857cc Mon Sep 17 00:00:00 2001 From: Ryan Houdek Date: Sun, 19 Nov 2023 01:25:33 -0800 Subject: [PATCH] FEXCore: Removes FEX_PACKED from CPUState Removes new warning from clang: ``` FEXCore/include/FEXCore/Core/CoreState.h:119:35: warning: not packing field 'DeferredSignalRefCount' as it is non-POD for the purposes of layout [-Wpacked-non-pod] 119 | NonAtomicRefCounter DeferredSignalRefCount; | ^ ``` We already ensure sane data layout that this is unnecessary anyway. --- FEXCore/include/FEXCore/Core/CoreState.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FEXCore/include/FEXCore/Core/CoreState.h b/FEXCore/include/FEXCore/Core/CoreState.h index e310f0c13..13859f292 100644 --- a/FEXCore/include/FEXCore/Core/CoreState.h +++ b/FEXCore/include/FEXCore/Core/CoreState.h @@ -72,7 +72,7 @@ namespace FEXCore::Core { static_assert(std::is_trivially_copyable_v>, "needs to be trivially copyable"); static_assert(sizeof(NonAtomicRefCounter) == sizeof(uint64_t), "Needs to be correct size"); - struct FEX_PACKED CPUState { + struct CPUState { // Allows more efficient handling of the register // file in the event AVX is not supported. union XMMRegs {