[PATCH] D125723: [MSVC] Add initial support for MSVC pragma optimize

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang/docs/LanguageExtensions.rst:3798
+``#pragma optimize("[optimization-list]", on | off)``. At the moment, Clang only
+supports an empty optimization list, whereas MSVC supports the parameters, ``s``,
+``g``, ``t``, and ``y``. Clang's current implementation of the pragma behaves in
----------------



================
Comment at: clang/docs/LanguageExtensions.rst:3799-3800
+supports an empty optimization list, whereas MSVC supports the parameters, ``s``,
+``g``, ``t``, and ``y``. Clang's current implementation of the pragma behaves in
+the same way as the clang pragma, ``#pragma clang optimize``. All functions
+between ``off`` and ``on`` will be decorated with the ``optnone`` attribute.
----------------
Would this be more clear?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125723/new/

https://reviews.llvm.org/D125723