New issue
Advanced search Search tips

Issue 782751 link

Starred by 1 user
Status: Fixed
Owner:
r...@chromium.org
Closed: Nov 2017
Cc:
h...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 782281



Sign in to add a comment

LLVM asserts in AArch64's generateCompactUnwindInfo implementation on opus/pitch.c

Project Member Reported by r...@chromium.org, Nov 8 2017 
First failing build: https://build.chromium.org/p/chromium.clang/builders/ToTiOS/builds/250

Window:
bad: 317603
good: 317568

There are several AArch64 .td file changes in that range, and one x86 CFI one which I initially suspected, but should be X86-only.

Comment 1 by h...@chromium.org, Nov 8 2017 

Got the source and invocation.
pitch-76b6c4.c
6.7 MB View Download
pitch-76b6c4.sh
7.9 KB View Download

Comment 2 by r...@chromium.org, Nov 8 2017 

There was no quick fix, so I had to revert it in r317726. It's a shame, though, because I really want accurate CFI and it's a big patch.

Comment 3 by r...@chromium.org, Nov 8 2017 

Reduction, for the curious:

int a, c, d, e, f, g, h, i, j, k, l, m;
void n(int o, int *b) {
  if (g)
    f = 0;
  for (; f < o; f++) {
    m = a;
    if (l > j * k > i)
      j = i = k = d;
    h = b[c] - e;
  }
}

Compile with `clang -c --target=arm64-apple-ios9.0.0 -O3`.

Comment 4 by h...@chromium.org, Nov 15 2017

Status: Fixed (was: Untriaged)
Let's call it fixed since the revert is in now.
►

Sign in to add a comment