exec: Poison Hexagon target-specific definitions

[PATCH] exec: Poison Hexagon target-specific definitions

Posted by Philippe Mathieu-Daudé 3 years, 2 months ago

Commit 3e7a84eeccc ("Hexagon build infrastructure") added Hexagon
definitions that should be poisoned on target independent device
code, but forgot to update "exec/poison.h". Do it now.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 include/exec/poison.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/exec/poison.h b/include/exec/poison.h
index d7ae1f23e71..6bb86f6c2f0 100644
--- a/include/exec/poison.h
+++ b/include/exec/poison.h
@@ -10,6 +10,7 @@
 #pragma GCC poison TARGET_ALPHA
 #pragma GCC poison TARGET_ARM
 #pragma GCC poison TARGET_CRIS
+#pragma GCC poison TARGET_HEXAGON
 #pragma GCC poison TARGET_HPPA
 #pragma GCC poison TARGET_LM32
 #pragma GCC poison TARGET_M68K
@@ -73,6 +74,7 @@
 #pragma GCC poison CONFIG_CRIS_DIS
 #pragma GCC poison CONFIG_HPPA_DIS
 #pragma GCC poison CONFIG_I386_DIS
+#pragma GCC poison CONFIG_HEXAGON_DIS
 #pragma GCC poison CONFIG_LM32_DIS
 #pragma GCC poison CONFIG_M68K_DIS
 #pragma GCC poison CONFIG_MICROBLAZE_DIS
-- 
2.26.2

RE: [PATCH] exec: Poison Hexagon target-specific definitions

Posted by Taylor Simpson 3 years, 2 months ago


> -----Original Message-----
> From: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com> On
> Behalf Of Philippe Mathieu-Daudé
> Sent: Friday, February 19, 2021 7:58 AM
> To: qemu-devel@nongnu.org; Richard Henderson
> <richard.henderson@linaro.org>
> Cc: Taylor Simpson <tsimpson@quicinc.com>; Philippe Mathieu-Daudé
> <f4bug@amsat.org>
> Subject: [PATCH] exec: Poison Hexagon target-specific definitions
>
> Commit 3e7a84eeccc ("Hexagon build infrastructure") added Hexagon
> definitions that should be poisoned on target independent device
> code, but forgot to update "exec/poison.h". Do it now.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  include/exec/poison.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/include/exec/poison.h b/include/exec/poison.h
> index d7ae1f23e71..6bb86f6c2f0 100644
> --- a/include/exec/poison.h
> +++ b/include/exec/poison.h
> @@ -10,6 +10,7 @@
>  #pragma GCC poison TARGET_ALPHA
>  #pragma GCC poison TARGET_ARM
>  #pragma GCC poison TARGET_CRIS
> +#pragma GCC poison TARGET_HEXAGON
>  #pragma GCC poison TARGET_HPPA
>  #pragma GCC poison TARGET_LM32
>  #pragma GCC poison TARGET_M68K
> @@ -73,6 +74,7 @@
>  #pragma GCC poison CONFIG_CRIS_DIS
>  #pragma GCC poison CONFIG_HPPA_DIS
>  #pragma GCC poison CONFIG_I386_DIS
> +#pragma GCC poison CONFIG_HEXAGON_DIS
>  #pragma GCC poison CONFIG_LM32_DIS
>  #pragma GCC poison CONFIG_M68K_DIS
>  #pragma GCC poison CONFIG_MICROBLAZE_DIS
> --
> 2.26.2

Reviewed-by: Taylor Simpson <tsimpson@quicinc.com>

Re: [PATCH] exec: Poison Hexagon target-specific definitions

Posted by Richard Henderson 3 years, 1 month ago

On 2/19/21 5:57 AM, Philippe Mathieu-Daudé wrote:
> Commit 3e7a84eeccc ("Hexagon build infrastructure") added Hexagon
> definitions that should be poisoned on target independent device
> code, but forgot to update "exec/poison.h". Do it now.
> 
> Signed-off-by: Philippe Mathieu-Daudé<f4bug@amsat.org>
> ---
>   include/exec/poison.h | 2 ++
>   1 file changed, 2 insertions(+)

Queued to hexagon-next.

r~