pd-xsample 0.3.2+git20170905.1.4441ae5-6 fails to build on ppc64el

Bug #2039810 reported by Benjamin Drung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pd-xsample (Debian)
Fix Released
Unknown
pd-xsample (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

pd-xsample 0.3.2+git20170905.1.4441ae5-6 fails to build on ppc64el:

g++ -DPD -I "/usr/include/pd" -DPD_FLOATSIZE=64 -DUNIX -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -fcheck-new -DNDEBUG -I/usr/include/flext -DFLEXT_SYS=2 -DFLEXT_SHARED -DFLEXT_USE_CMEM -pthread -I/usr/include/pd -DPD -Wdate-time -D_FORTIFY_SOURCE=2 -g -O3 -ffile-prefix-map=/<<PKGBUILDDIR>>=. -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -fdebug-prefix-map=/<<PKGBUILDDIR>>=/usr/src/pd-xsample-0.3.2+git20170905.1.4441ae5-6 -o source/record.linux-ppc64l-64.so.o -c source/record.cpp
[...]
source/groove.cpp:358:16: error: invalid parameter combination for AltiVec intrinsic ‘__builtin_vec_dst’
  358 | vec_dst(speed,GetPrefetchConstant(1,n>>2,0),0);
      | ^

Full log: https://launchpadlibrarian.net/683921963/buildlog_ubuntu-mantic-ppc64el.pd-xsample_0.3.2+git20170905.1.4441ae5-6_BUILDING.txt.gz

Setting `PD_FLOATSIZE=64` will lead to the `speed` type `t_sample` being defined as `double`. According to https://gcc.gnu.org/onlinedocs/gcc/PowerPC-AltiVec-Built-in-Functions-on-ISA-2_002e05.html vec_dst is defined for `float` but not for `double`.

"Build Pd64 externals" was the only change of pd-xsample 0.3.2+git20170905.1.4441ae5-6. So let's skip this change for ppc64el.

Changed in pd-xsample (Debian):
status: Unknown → Fix Released
Revision history for this message
Benjamin Drung (bdrung) wrote :

Uploaded 0.3.2+git20170905.1.4441ae5-7 to Debian unstable to address the build failure.

Changed in pd-xsample (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pd-xsample - 0.3.2+git20170905.1.4441ae5-7

---------------
pd-xsample (0.3.2+git20170905.1.4441ae5-7) unstable; urgency=medium

  * Team upload.
  * Skip Pd64 externals for ppc64el, because vec_dst does not support double
    (Closes: #1050828, LP: #2039810)

 -- Benjamin Drung <email address hidden> Thu, 19 Oct 2023 15:52:10 +0200

Changed in pd-xsample (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.