New issue
Advanced search Search tips

Issue 801303 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2018
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 800984



Sign in to add a comment

dev-libs/libffi fails to build with llvm-next

Project Member Reported by rahulchaudhry@chromium.org, Jan 11 2018

Issue description

This happens when testing the next version of llvm: https://bugs.chromium.org/p/chromium/issues/detail?id=800984

For daisy (arm), dev-libs/libffi build fails with new warnings and errors.

The new warnings are from [-Wunused-result].
Here is one example:
../libffi-3.1/src/closures.c:460:7: warning: ignoring return value of function declared with 'warn_unused_result' attribute [-Wunused-result]
      ftruncate (execfd, offset);
      ^~~~~~~~~  ~~~~~~~~~~~~~~


The new errors are from unrecognized instructions in assembly files.
Here are a couple of examples:
/build/daisy/tmp/portage/dev-libs/libffi-3.1-r4/work/libffi-3.1/src/arm/sysv.S:363:2: error: invalid instruction, did you mean: fldmiax?
 fldmiadgt ip, {d0-d7}
 ^
/build/daisy/tmp/portage/dev-libs/libffi-3.1-r4/work/libffi-3.1/src/arm/sysv.S:412:2: error: invalid instruction, did you mean: fstmiax?
 fstmiadeq r2, {d0-d3}
 ^

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/b16daa5195317cab0e77047b756e73ccb74e987f

commit b16daa5195317cab0e77047b756e73ccb74e987f
Author: Rahul Chaudhry <rahulchaudhry@chromium.org>
Date: Sat Jan 20 04:35:39 2018

dev-libs/libffi: fix build issue with llvm-next.

Arm assembly syntax used in libffi is not recognized by
the integrated assembler.

BUG= chromium:801303 
TEST='emerge-daisy dev-libs/libffi' works with llvm-next.

Change-Id: Ifcccc185c1cf6a97c47047a0f9346fa4a50adac8
Reviewed-on: https://chromium-review.googlesource.com/876766
Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org>
Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>

[rename] https://crrev.com/b16daa5195317cab0e77047b756e73ccb74e987f/dev-libs/libffi/libffi-3.1-r5.ebuild

Status: Fixed (was: Untriaged)

Sign in to add a comment