Nikunj A Dadhania <nik...@linux.vnet.ibm.com> writes:
> Peter Maydell <peter.mayd...@linaro.org> writes:
>
>> On 22 September 2016 at 07:36, David Gibson <da...@gibson.dropbear.id.au>
>> wrote:
>>> The following changes since commit a008535b9fa396226ff9cf78b8ac5f3584bda58e:
>>>
>>> build-sys: fix make install regression (2016-09-20 11:32:43 +0100)
>>>
>>> are available in the git repository at:
>>>
>>> git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20160922
>>>
>>> for you to fetch changes up to 2832da4b6fc549d5feb2cf9fe53ad98cee894327:
>>>
>>> monitor: fix crash for platforms without a CPU 0 (2016-09-22 15:53:01
>>> +1000)
>>>
>>> ----------------------------------------------------------------
>>> ppc patch queue 2016-09-22
>>>
>>> This is my second pull request of ppc and spapr related patches for
>>> qemu-2.8. Included here are
>>> * TCG implementations for more POWER9 instructions
>>> * Some preliminary XICS fixes in preparataion for the pnv machine type
>>> * A significant ADB (Macintosh kbd/mouse) cleanup
>>> * Some conversions to use trace instead of debug macros
>>> * Fixes to correctly handle global TLB flush synchronization in
>>> TCG. This is already a bug, but it will have much more impact
>>> when we get MTTCG
>>> * Add more qtest testcases for Power
>>> * Some MAINTAINERS updates
>>> * Assorted bugfixes
>>>
>>> This touches some test files and monitor.c which are technically
>>> outside the ppc code, but coming through this tree because the changes
>>> are primarily of interest to ppc.
>>>
>>> ----------------------------------------------------------------
>>
>> I'm afraid this fails to build with clang:
>>
>> /home/petmay01/linaro/qemu-for-merges/target-ppc/translate.c:532:16:
>> error: unused function 'L' [-Werro
>> r,-Wunused-function]
>> EXTRACT_HELPER(L, 16, 2);
>> ^
>> 1 error generated.
>
> +/* darn */
> +static void gen_darn(DisasContext *ctx)
> +{
> + int l = L(ctx->opcode);
>
> Used here, false positive ?
It is not a false positive, below would be needed in "[PULL 34/44]
target-ppc: implement darn instruction" as gen_darn is within
TARGET_PPC64:
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index eb681de..4f33915 100644
--- a/target-ppc/translate.c
+++ b/target-ppc/translate.c
@@ -528,8 +528,10 @@ EXTRACT_HELPER(FPW, 16, 1);
/* addpcis */
EXTRACT_HELPER_DXFORM(DX, 10, 6, 6, 5, 16, 1, 1, 0, 0)
+#if defined(TARGET_PPC64)
/* darn */
EXTRACT_HELPER(L, 16, 2);
+#endif
/*** Jump target decoding ***/
/* Immediate address */
Regards,
Nikunj
