docs/devel/tcg-ops: Bury mentions of trunc_shr_i64_i32()

[PATCH] docs/devel/tcg-ops: Bury mentions of trunc_shr_i64_i32()

Philippe Mathieu-Daudé posted 1 patch 8 months, 4 weeks ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230822162847.71206-1-philmd@linaro.org

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>

docs/devel/tcg-ops.rst | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH] docs/devel/tcg-ops: Bury mentions of trunc_shr_i64_i32()

Posted by Philippe Mathieu-Daudé 8 months, 4 weeks ago

Commit 609ad70562 ("tcg: Split trunc_shr_i32 opcode into
extr[lh]_i64_i32") remove trunc_shr_i64_i32(). Update the
backend documentation.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 docs/devel/tcg-ops.rst | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/docs/devel/tcg-ops.rst b/docs/devel/tcg-ops.rst
index 6a166c5665..53695e1623 100644
--- a/docs/devel/tcg-ops.rst
+++ b/docs/devel/tcg-ops.rst
@@ -882,14 +882,15 @@ sub2_i32, brcond2_i32).
 On a 64 bit target, the values are transferred between 32 and 64-bit
 registers using the following ops:
 
-- trunc_shr_i64_i32
+- extrl_i64_i32
+- extrh_i64_i32
 - ext_i32_i64
 - extu_i32_i64
 
 They ensure that the values are correctly truncated or extended when
 moved from a 32-bit to a 64-bit register or vice-versa. Note that the
-trunc_shr_i64_i32 is an optional op. It is not necessary to implement
-it if all the following conditions are met:
+extrl_i64_i32 and extrh_i64_i32 are optional ops. It is not necessary
+to implement them if all the following conditions are met:
 
 - 64-bit registers can hold 32-bit values
 - 32-bit values in a 64-bit register do not need to stay zero or
-- 
2.41.0

Re: [PATCH] docs/devel/tcg-ops: Bury mentions of trunc_shr_i64_i32()

Posted by Richard Henderson 8 months, 4 weeks ago

On 8/22/23 09:28, Philippe Mathieu-Daudé wrote:
> Commit 609ad70562 ("tcg: Split trunc_shr_i32 opcode into
> extr[lh]_i64_i32") remove trunc_shr_i64_i32(). Update the
> backend documentation.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   docs/devel/tcg-ops.rst | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH] docs/devel/tcg-ops: Bury mentions of trunc_shr_i64_i32()

Posted by Richard Henderson 8 months, 4 weeks ago

On 8/22/23 10:08, Richard Henderson wrote:
> On 8/22/23 09:28, Philippe Mathieu-Daudé wrote:
>> Commit 609ad70562 ("tcg: Split trunc_shr_i32 opcode into
>> extr[lh]_i64_i32") remove trunc_shr_i64_i32(). Update the
>> backend documentation.
>>
>> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
>> ---
>>   docs/devel/tcg-ops.rst | 7 ++++---
>>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> 
> r~

Queued to tcg-next.


r~