Skip to content

Add cmake option to enable microprofile#651

Merged
OpenSauce04 merged 1 commit into
azahar-emu:masterfrom
PabloMK7:disablemicroprofile
Mar 10, 2025
Merged

Add cmake option to enable microprofile#651
OpenSauce04 merged 1 commit into
azahar-emu:masterfrom
PabloMK7:disablemicroprofile

Conversation

@PabloMK7

@PabloMK7 PabloMK7 commented Mar 9, 2025

Copy link
Copy Markdown
Member

Disables microprofile by default, saving a bit of RAM. Add a cmake compile time define instead.

@PabloMK7

PabloMK7 commented Mar 9, 2025

Copy link
Copy Markdown
Member Author

Currently running CI with the option enabled by default, if CI succeeds will do the opposite.

@PabloMK7 PabloMK7 marked this pull request as draft March 9, 2025 18:39
@PabloMK7 PabloMK7 added this to the 2120 (Azahar initial release) milestone Mar 9, 2025
@PabloMK7 PabloMK7 force-pushed the disablemicroprofile branch from e03e388 to 6794161 Compare March 9, 2025 19:10
@PabloMK7 PabloMK7 marked this pull request as ready for review March 9, 2025 19:10
@PabloMK7

PabloMK7 commented Mar 9, 2025

Copy link
Copy Markdown
Member Author

CI succeeded with option enabled, trying the other way around

@PabloMK7 PabloMK7 force-pushed the disablemicroprofile branch from 6794161 to 8ad116b Compare March 9, 2025 19:20
@PabloMK7 PabloMK7 requested a review from OpenSauce04 March 9, 2025 19:43
Comment thread src/video_core/renderer_opengl/gl_resource_manager.h
Comment thread externals/CMakeLists.txt
Comment thread externals/CMakeLists.txt
@OpenSauce04 OpenSauce04 added the enhancement New feature or request label Mar 9, 2025
@PabloMK7 PabloMK7 requested a review from OpenSauce04 March 10, 2025 09:07
@PabloMK7 PabloMK7 force-pushed the disablemicroprofile branch from 8ad116b to 7062c61 Compare March 10, 2025 10:59
@OpenSauce04 OpenSauce04 merged commit dd9c743 into azahar-emu:master Mar 10, 2025
@PabloMK7 PabloMK7 deleted the disablemicroprofile branch March 10, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants