[Qemu-devel] [PATCH 07/13] target: arm: Remove the LDST headers

Samuel Ortiz posted 13 patches 6 years, 11 months ago
There is a newer version of this series
[Qemu-devel] [PATCH 07/13] target: arm: Remove the LDST headers
Posted by Samuel Ortiz 6 years, 11 months ago
They are no longer needed.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Robert Bradford <robert.bradford@intel.com>
---
 target/arm/helper.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/target/arm/helper.c b/target/arm/helper.c
index 689879c23a..dcb689bfbb 100644
--- a/target/arm/helper.c
+++ b/target/arm/helper.c
@@ -16,7 +16,6 @@
 #include "sysemu/sysemu.h"
 #include "qemu/crc32c.h"
 #include "exec/exec-all.h"
-#include "arm_ldst.h"
 #include <zlib.h> /* For crc32 */
 #include "exec/semihost.h"
 #include "sysemu/kvm.h"
-- 
2.19.1


Re: [Qemu-devel] [PATCH 07/13] target: arm: Remove the LDST headers
Posted by Peter Maydell 6 years, 11 months ago
On 13 November 2018 at 16:52, Samuel Ortiz <sameo@linux.intel.com> wrote:
> They are no longer needed.

...because a previous patch moved the code using them out
to excp_helper.c.

> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Reviewed-by: Robert Bradford <robert.bradford@intel.com>
> ---
>  target/arm/helper.c | 1 -
>  1 file changed, 1 deletion(-)

thanks
-- PMM