[mythtv-commits] [MythTV/mythtv] 6a7908: configure: arm: Don't add -march= to the compiler ...

Martin Storsjö noreply at github.com
Fri Feb 25 01:06:14 UTC 2022


  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 6a79086179f0ffc6572285bfae4875a4db9a8610
      https://github.com/MythTV/mythtv/commit/6a79086179f0ffc6572285bfae4875a4db9a8610
  Author: Martin Storsjö <martin at martin.st>
  Date:   2022-02-24 (Thu, 24 Feb 2022)

  Changed paths:
    M mythtv/external/FFmpeg/configure

  Log Message:
  -----------
  configure: arm: Don't add -march= to the compiler if no preference was passed

If no --cpu= option was passed to configure, we detect what the
compiler defaults to. This detected value was then fed back to the
rest of the configure logic, as if it was an explicit choice.

This breaks on Ubuntu 21.10 with GCC 11.1.

Since GCC 8, it's possible to add configure extra features via the
-march option, like e.g. -march=armv7-a+neon. If the -mfpu= option
is configured to default to 'auto', the fpu setting gets taken
from the -march option.

GCC 11.1 in Ubuntu seems to be configured to use -mfpu=auto. This
has the effect of breaking any compilation command that specifies
-march=armv7-a, because the driver implicitly also adds -mfloat-abi=hard,
and that combination results in this error:

    cc1: error: ‘-mfloat-abi=hard’: selected processor lacks an FPU

One can compile successfully by passing e.g. -march=armv7-a+fp.

Therefore, restructure configure. If no specific preference was set
(and the 'cpu' configure variable was set as the output of
probe_arm_arch), the value we tried to set via -march= was the same
value that we just tried to detect as the compiler default.

So instead, just try to detect what the compiler defaults to, with
to allow setting other configure settings (such as 'fast_unaligned'),
but don't try to spell out the compiler's default via the -march flag.

Signed-off-by: Peter Bennett <pbennett at mythtv.org>




More information about the mythtv-commits mailing list