On Wed, Jan 4, 2017 at 3:20 PM, George Kyriazis
<george.kyria...@intel.com> wrote:
> wrap lp_bld_type.h around extern "C".
> Windows decorates global variables, so when used from .cpp files, need
> to use an undecorated version.
> ---
> src/gallium/auxiliary/gallivm/lp_bld_type.h | 7 +++++++
> src/gallium/drivers/swr/swr_screen.cpp | 4 ----
> 2 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_type.h
> b/src/gallium/auxiliary/gallivm/lp_bld_type.h
> index 7fb449f..afe8722 100644
> --- a/src/gallium/auxiliary/gallivm/lp_bld_type.h
> +++ b/src/gallium/auxiliary/gallivm/lp_bld_type.h
> @@ -41,6 +41,10 @@
> #include "pipe/p_compiler.h"
> #include "gallivm/lp_bld.h"
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> /**
> * Native SIMD architecture width available at runtime.
> *
> @@ -449,5 +453,8 @@ lp_build_context_init(struct lp_build_context *bld,
> unsigned
> lp_build_count_ir_module(LLVMModuleRef module);
>
> +#ifdef __cplusplus
> +}
> +#endif
>
> #endif /* !LP_BLD_TYPE_H */
Changes to this file are
Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>
> diff --git a/src/gallium/drivers/swr/swr_screen.cpp
> b/src/gallium/drivers/swr/swr_screen.cpp
> index a9905d7..3b010fe 100644
> --- a/src/gallium/drivers/swr/swr_screen.cpp
> +++ b/src/gallium/drivers/swr/swr_screen.cpp
> @@ -39,10 +39,6 @@
>
> #include "state_tracker/sw_winsys.h"
>
> -extern "C" {
> -#include "gallivm/lp_bld_limits.h"
> -}
> -
I'm still confused by this hunk. It doesn't seem to be related to the
change description. It also doesn't define any function prototypes, so
it doesn't really need the extern "C" thing either.
> #include "jit_api.h"
>
> #include "memory/TilingFunctions.h"
> --
> 2.10.0.windows.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
