Re: [Mesa-dev] [PATCH v1 1/2] i965/tools: 32bit compilation with meson

2018-09-06 Thread Lionel Landwerlin 

And pushed.

On 06/09/2018 11:53, Lionel Landwerlin wrote:

Oh right, I see, this is the only using gen_invalidate_range().

Thanks!

Reviewed-by: Lionel Landwerlin 

On 06/09/2018 11:40, Sergii Romantsov wrote:

With -Dtools=all compilation is passed now.
At this moment looking additionally on linkage issue with building, 
have solution but would like to find better, so hope will provide 
additional 3rd commit today.


On Thu, Sep 6, 2018 at 1:03 PM, Lionel Landwerlin 
> wrote:


Aren't other tools affected as well?

Thanks,

-
Lionel

On 06/09/2018 10:59, Sergii Romantsov wrote:

Building of 32bit mesa with meson causes issue:
"implicit declaration of function ‘__builtin_ia32_clflush’".
Fixed by adding msse2 compilation flag.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107843

Fixes: 314879f7fec0 (i965: Fix asynchronous mappings on !LLC
platforms.)
Signed-off-by: Sergii Romantsov
mailto:sergii.romant...@globallogic.com>>
---
  src/intel/tools/meson.build | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/tools/meson.build
b/src/intel/tools/meson.build
index 3617478..44d6bdd 100644
--- a/src/intel/tools/meson.build
+++ b/src/intel/tools/meson.build
@@ -90,7 +90,7 @@ libintel_sanitize_gpu = shared_library(
    dependencies : [dep_dl, dep_thread],
    include_directories : [inc_common, inc_intel, inc_drm_uapi],
    link_with : [libintel_common, libmesa_util],
-  c_args : [c_vis_args, no_override_init_args],
+  c_args : [c_vis_args, no_override_init_args, c_sse2_args],
    build_by_default : true,
    install_dir: get_option('libexecdir'),
    install: true



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org

https://lists.freedesktop.org/mailman/listinfo/mesa-dev





--
Sergii Romantsov
GlobalLogic Inc.
www.globallogic.com 




___
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

Re: [Mesa-dev] [PATCH v1 1/2] i965/tools: 32bit compilation with meson

2018-09-06 Thread Lionel Landwerlin 

Oh right, I see, this is the only using gen_invalidate_range().

Thanks!

Reviewed-by: Lionel Landwerlin 

On 06/09/2018 11:40, Sergii Romantsov wrote:

With -Dtools=all compilation is passed now.
At this moment looking additionally on linkage issue with building, 
have solution but would like to find better, so hope will provide 
additional 3rd commit today.


On Thu, Sep 6, 2018 at 1:03 PM, Lionel Landwerlin 
mailto:lionel.g.landwer...@intel.com>> 
wrote:


Aren't other tools affected as well?

Thanks,

-
Lionel

On 06/09/2018 10:59, Sergii Romantsov wrote:

Building of 32bit mesa with meson causes issue:
"implicit declaration of function ‘__builtin_ia32_clflush’".
Fixed by adding msse2 compilation flag.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107843

Fixes: 314879f7fec0 (i965: Fix asynchronous mappings on !LLC
platforms.)
Signed-off-by: Sergii Romantsov
mailto:sergii.romant...@globallogic.com>>
---
  src/intel/tools/meson.build | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/tools/meson.build
b/src/intel/tools/meson.build
index 3617478..44d6bdd 100644
--- a/src/intel/tools/meson.build
+++ b/src/intel/tools/meson.build
@@ -90,7 +90,7 @@ libintel_sanitize_gpu = shared_library(
    dependencies : [dep_dl, dep_thread],
    include_directories : [inc_common, inc_intel, inc_drm_uapi],
    link_with : [libintel_common, libmesa_util],
-  c_args : [c_vis_args, no_override_init_args],
+  c_args : [c_vis_args, no_override_init_args, c_sse2_args],
    build_by_default : true,
    install_dir: get_option('libexecdir'),
    install: true



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org 
https://lists.freedesktop.org/mailman/listinfo/mesa-dev





--
Sergii Romantsov
GlobalLogic Inc.
www.globallogic.com 



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v1 1/2] i965/tools: 32bit compilation with meson

2018-09-06 Thread Sergii Romantsov 
With -Dtools=all compilation is passed now.
At this moment looking additionally on linkage issue with building, have
solution but would like to find better, so hope will provide additional 3rd
commit today.

On Thu, Sep 6, 2018 at 1:03 PM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:

> Aren't other tools affected as well?
>
> Thanks,
>
> -
> Lionel
>
> On 06/09/2018 10:59, Sergii Romantsov wrote:
>
>> Building of 32bit mesa with meson causes issue:
>> "implicit declaration of function ‘__builtin_ia32_clflush’".
>> Fixed by adding msse2 compilation flag.
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107843
>> Fixes: 314879f7fec0 (i965: Fix asynchronous mappings on !LLC platforms.)
>> Signed-off-by: Sergii Romantsov 
>> ---
>>   src/intel/tools/meson.build | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/intel/tools/meson.build b/src/intel/tools/meson.build
>> index 3617478..44d6bdd 100644
>> --- a/src/intel/tools/meson.build
>> +++ b/src/intel/tools/meson.build
>> @@ -90,7 +90,7 @@ libintel_sanitize_gpu = shared_library(
>> dependencies : [dep_dl, dep_thread],
>> include_directories : [inc_common, inc_intel, inc_drm_uapi],
>> link_with : [libintel_common, libmesa_util],
>> -  c_args : [c_vis_args, no_override_init_args],
>> +  c_args : [c_vis_args, no_override_init_args, c_sse2_args],
>> build_by_default : true,
>> install_dir: get_option('libexecdir'),
>> install: true
>>
>
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>



-- 
Sergii Romantsov
GlobalLogic Inc.
www.globallogic.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v1 1/2] i965/tools: 32bit compilation with meson

2018-09-06 Thread Lionel Landwerlin 

Aren't other tools affected as well?

Thanks,

-
Lionel

On 06/09/2018 10:59, Sergii Romantsov wrote:

Building of 32bit mesa with meson causes issue:
"implicit declaration of function ‘__builtin_ia32_clflush’".
Fixed by adding msse2 compilation flag.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107843
Fixes: 314879f7fec0 (i965: Fix asynchronous mappings on !LLC platforms.)
Signed-off-by: Sergii Romantsov 
---
  src/intel/tools/meson.build | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/tools/meson.build b/src/intel/tools/meson.build
index 3617478..44d6bdd 100644
--- a/src/intel/tools/meson.build
+++ b/src/intel/tools/meson.build
@@ -90,7 +90,7 @@ libintel_sanitize_gpu = shared_library(
dependencies : [dep_dl, dep_thread],
include_directories : [inc_common, inc_intel, inc_drm_uapi],
link_with : [libintel_common, libmesa_util],
-  c_args : [c_vis_args, no_override_init_args],
+  c_args : [c_vis_args, no_override_init_args, c_sse2_args],
build_by_default : true,
install_dir: get_option('libexecdir'),
install: true



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH v1 1/2] i965/tools: 32bit compilation with meson

2018-09-06 Thread Sergii Romantsov 
Building of 32bit mesa with meson causes issue:
"implicit declaration of function ‘__builtin_ia32_clflush’".
Fixed by adding msse2 compilation flag.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107843
Fixes: 314879f7fec0 (i965: Fix asynchronous mappings on !LLC platforms.)
Signed-off-by: Sergii Romantsov 
---
 src/intel/tools/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/tools/meson.build b/src/intel/tools/meson.build
index 3617478..44d6bdd 100644
--- a/src/intel/tools/meson.build
+++ b/src/intel/tools/meson.build
@@ -90,7 +90,7 @@ libintel_sanitize_gpu = shared_library(
   dependencies : [dep_dl, dep_thread],
   include_directories : [inc_common, inc_intel, inc_drm_uapi],
   link_with : [libintel_common, libmesa_util],
-  c_args : [c_vis_args, no_override_init_args],
+  c_args : [c_vis_args, no_override_init_args, c_sse2_args],
   build_by_default : true,
   install_dir: get_option('libexecdir'),
   install: true
-- 
2.7.4

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev