pitrou commented on code in PR #35924: URL: https://github.com/apache/arrow/pull/35924#discussion_r1234965165
########## docs/source/developers/cpp/building.rst: ########## @@ -254,6 +254,34 @@ Several build types are possible: * ``Release``: applies compiler optimizations and removes debug information from the binary. +.. note:: + + These built types provide suitable optimization/debug flags by + default but you can change them by specifying + ``-DARROW_C_FLAGS_${BUILD_TYPE}=...`` and/or + ``-DARROW_CXX_FLAGS_${BUILD_TYPE}=...``. ``${BUILD_TYPE}`` is upper + case of build type. For example, ``DEBUG`` + (``-DARROW_C_FLAGS_DEBUG=...`` / ``-DARROW_CXX_FLAGS_DEBUG=...``) for the + ``Debug`` build type and ``RELWITHDEBINFO`` + (``-DARROW_C_FLAGS_RELWITHDEBINFO=...`` / + ``-DARROW_CXX_FLAGS_RELWITHDEBINFO=...``) for the ``RelWithDebInfo`` + build type. + + For example, you can use ``-O3`` as an optimization flag by + specifying ``-DARROW_CXX_FLAGS_RELEASE=-O3`` for the ``Release`` + build type. You can use ``-g3`` as a debug flag by specifying + ``-DARROW_CXX_FLAGS_DEBUG=-g3`` for the ``Debug`` build type. Review Comment: ```suggestion For example, you can use ``-O3`` as an optimization flag for the ``Release`` build type by passing ``-DARROW_CXX_FLAGS_RELEASE=-O3`` . You can use ``-g3`` as a debug flag for the ``Debug`` build type by passing ``-DARROW_CXX_FLAGS_DEBUG=-g3`` . ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org