[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] x86emul: replace further UB shifts
- To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- Date: Tue, 4 Aug 2020 14:19:51 +0100
- Authentication-results: esa1.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
- Cc: Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Delivery-date: Tue, 04 Aug 2020 13:20:05 +0000
- Ironport-sdr: mmZ1w8wGMdgcbwsEJkHf78wDVmy4D8SLLB/y6+di9wLsIPRbS3X+ekvsHWPS4bRYuIlCzjiev7 RwGcZ2aCv2Luv9sEH5PkFv0FA1fGZlAEuS1lkpGBHSNJ8gtp3Ynxa6yYHfVr5r6MmMmmq3ellt xTap3NlACrHUwTGA4kFa5eC8eG2WgcwyACqDPbgf6BhdvFkW3hb+OeLoUD4PS3zspXm0cWrnnw 4HW8CjwualxsnSiErEB9dfEC/LtAHzYJujmN9XsnZq2OwF7qbudI/wwCCG9Kkr7yQBLaRudOH9 6Yw=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 04/08/2020 11:13, Jan Beulich wrote:
> I have no explanation how I managed to overlook these while putting
> together what is now b6a907f8c83d ("x86emul: replace UB shifts").
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
I've just checked, and GCC 9.3.0 does convert this pattern back to a
single shift instruction, so at least we're not left with real mul
instructions.
~Andrew
|