tree 47eced60f7e5e8c3921c72b7329f9d23384b7f59
parent eac32c45dd443c1ae42622c2ef70a3c4ce7896a7
author Ben Clayton <bclayton@google.com> 1556202163 +0100
committer Ben Clayton <bclayton@google.com> 1556297504 +0000

CMakeLists: Fix REACTOR_EMIT_DEBUG_INFO builds

ENABLE_RR_DEBUG_INFO was not being passed to the ReactorLLVM target.
This introduced a subtle calling mismatch as the RValue<T> copy constructor is conditionally added only when ENABLE_RR_DEBUG_INFO is defined.
This would cause parameters to go out of sync for inlined functions (not entirely sure I understand the underlying cause here), leading to hilarity.

This change also moves "-fno-exceptions" back to the global cpp_flags. This is a true global setting and should not have been moved to SWIFTSHADER_COMPILE_OPTIONS.

Change-Id: Ib20777f1ee4b443358ada667902f9a6a1ac3513a
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29932
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>