https://bugs.kde.org/show_bug.cgi?id=368412

--- Comment #5 from Will Schmidt <will_schm...@vnet.ibm.com> ---
(In reply to Mark Wielaard from comment #3)
> You are changing the AC_DEFINE from HAS_ALTIVEC to COMPILER_SUPPORTS_ALTIVEC.
> But COMPILER_SUPPORTS_ALTIVEC is never used as C preprocessor symbol, so I
> think you should just remove it.

Yup, agreed, done.    Thanks for the review.

> HAS_ALTIVEC is used as an C preprocessor symbol in
> none/tests/ppc32/jm-insns.c but there it is defined in the Makefile
> depending on the AM_CONDITIONAL HAS_ALTIVEC. So that should be fine

Yup, ..  I had added the AM_CONDITIONAL way back in 15424, but I obviously
missed removing the old definition, which caused the problem..

> (although the comment in the code implies it comes from config.h, which
> isn't true anymore).

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to