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

Remove embedded per-file options in a few projects #13

Conversation

clayne
Copy link
Contributor

@clayne clayne commented Aug 14, 2023

  • OpenAL: 4 source files were forcing MaxSpeed. They should instead respect the configuration they use.

  • xrCPU_Pipe: 1 source file (an SSE specific one with embedded asm) is forcing MaxSpeed and various preprocessor options which appear left over from when the inline assembly was being written/debugged. The config will set the appropriate optimization level.

    This was noticed because VerifiedDX11 is set to no optimization for debugging purposes but optimization kept being applied to certain files.

  • xrCore: Remove AssemblerOutput from a few files which appears to be left over from debugging. The .asm file isn't used by anything but humans.

* OpenAL: 4 source files were forcing MaxSpeed. They should instead
  respect the configuration they use.

* xrCPU_Pipe: 1 source file (an SSE specific one with embedded asm) is
  forcing MaxSpeed and various preprocessor options which appear left
  over from when the inline assembly was being written/debugged. The
  config will set the appropriate optimization level.

  This was noticed because VerifiedDX11 is set to no optimization for
  debugging purposes but optimization kept being applied to certain
  files.

* xrCore: Remove AssemblerOutput from a few files which appears to be
  left over from debugging. The .asm file isn't used by anything but
  humans.
@themrdemonized themrdemonized merged commit 5eefcb9 into themrdemonized:all-in-one-vs2022 Aug 15, 2023
themrdemonized added a commit that referenced this pull request Aug 15, 2023
…s-cleanup

Remove embedded per-file options in a few projects

(cherry picked from commit 5eefcb9)
@clayne clayne deleted the 1692032080-vcxproj-embedded-options-cleanup branch August 19, 2023 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants