New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Cc:
Components:



Sign in to add a comment
link

Issue 4384: Subzero Compilation Fails with error

Reported by dtipp...@pdftron.com, Dec 23 2016

Issue description

What version of the SDK are you using? 41
What version of the runtime are you using? Not sure? It seems we can't easily obtain the NaCl version from Chrome?
On what operating system? Windows 10 64-bit 1607 14393.576
What browser are you using? Chrome 55.0.2883.87

What steps will reproduce the problem?
The problem is encountered when using subzero (using optlevel:0) to compile the .pexe zipped at http://pdftron.s3.amazonaws.com/custom/ID-zJWLuhTffd3c/google/PDFWorkerFull.zip.

What is the expected output? 

Compilation is successful etc.

What do you see instead?

NativeClient: PnaclCoordinator: Compile stream end failed: reply not received from PNaCl translator (it probably crashed)
 

Comment 1 by bugdroid1@chromium.org, Jan 4 2017

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/native_client/src/native_client.git/+/01d3ae00d277726b8aaee572120a5ba2a31d511e

commit 01d3ae00d277726b8aaee572120a5ba2a31d511e
Author: Jim Stichnoth <stichnot@chromium.org>
Date: Wed Jan 04 07:04:41 2017

PNaCl: Update subzero revision in pnacl/COMPONENT_REVISIONS

This pulls in the following Subzero changes:

ec92917: (jaydeep.patil@imgtec.com) [SubZero] Legalize load, store for MIPS post lower
4683237: (sagar.thakur@imgtec.com) [Subzero][MIPS32] Fix alloca alignment and offset for Om1 and O2 optimization
45e4d5e: (jaydeep.patil@imgtec.com) [SubZero] Handle relocatable constants for MIPS
3a01f33: (jaydeep.patil@imgtec.com) [SubZero] Implement Fcmp, ICmp, Cast and Select for vector type
f52cea4: (capn@google.com) Fix unpacking from a single vector.
7638e27: (capn@google.com) Add x86 vector packing instructions.
0e90622: (capn@google.com) Generate error on unexpected intrisics.
89be887: (sagar.thakur@imgtec.com) [Subzero][MIPS32] Account for variable alloca alignment bytes in addProlog
1448d95: (capn@google.com) Optimize shuffles corresponding to x86 punpckh instructions.
7145e69: (stichnot@chromium.org) Subzero: Fix compiler warnings.
32f9cce: (capn@google.com) Fix 64-bit pointer type for non-x32 ABIs.
ef8210d: (capn@google.com) Implement vector packing intrinsics.
8b8af82: (capn@google.com) Implement bitcast between i32 and (emulated) v4i8.
e3cabda: (capn@google.com) Implement vector sign mask intrinsic.
d0e3030: (capn@google.com) Assert that PNaCl bitcode only uses 128-bit vector casts.
c9e91af: (capn@google.com) Remove verified asserts.
61593fb: (capn@google.com) Fix unit tests.
a3688ea: (capn@google.com) Fix two-vector unpack case.
a7979bf: (jaydeep.patil@imgtec.com) [SubZero] Fix f64 to/from i64 moves
0dabe18: (makdstefan@gmail.com) Subzero, MIPS32: Remove --skip-unimplemented from lit tests
130aca7: (jaydeep.patil@imgtec.com) [SubZero] Generate relocations for MIPS
73ae4fd: (capn@google.com) Preserve rsi and rdi when using Microsoft x86-64 calling convention.
7ad028e: (makdstefan@gmail.com) This patch enables running a couple more of lit tests for MIPS32
956cfd6: (capn@google.com) Generalize the Sqrt intrinsic to process vectors.
13cde0f: (capn@google.com) Implement integer vector multiply intrinsics.
67a49b5: (capn@google.com) Implement saturated vector add/subtract.
0c4c07d: (jaydeep.patil@imgtec.com) [SubZero] Fix code generation for vector type
f8c9977: (makdstefan@gmail.com) Subzero, MIPS32: Stacksave/Stackrestore implementation
f0d12c3: (capn@google.com) Implement floating-point rounding intrinsic.
3da9f65: (jaydeep.patil@imgtec.com) [SubZero] Generate MIPS.abiflags section
21f78bb: (jaydeep.patil@imgtec.com) [SubZero] Utilize instructions with immediate operands
83425de: (capn@google.com) Support 64-bit jump tables with LP64 data model.
6e03343: (makdstefan@gmail.com) Subzero, MIPS32: Sandbox initial patch
becb85f: (sagar.thakur@imgtec.com) [Subzero][MIPS] Implements atomic intrinsics for MIPS32
8208e75: (makdstefan@gmail.com) Subzero, MIPS32: Changes for improving sandbox crosstest results
2220990: (capn@google.com) Fix offset adjustment in x86 address optimization.
ef18fc5: (capn@google.com) Match sub-vector load/store operand order to regular load/store.
8be6975: (jaydeep.patil@imgtec.com) [SubZero] Fix size of arguments on stack
e1e1783: (capn@google.com) Fix skipping deleted instructions before replacing operands.
a29da90: (capn@google.com) Ensure that the sub-vector load destination is a register.
579b1b3: (capn@google.com) Generalize vector shuffling to accept any operand.
373913f: (stichnot@chromium.org) Subzero: Legalize the movzx argument.
35bbca3: (stichnot@chromium.org) Subzero: Fix multiply defined symbols in Windows/g++ build.

BUG=  swiftshader:24 
BUG=  swiftshader:9 
BUG= swiftshader:22
BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4384
BUG=  swiftshader:15 
TEST= PNaCl toolchain trybots
R=dschuff@chromium.org

Review-Url: https://codereview.chromium.org/2602613002 .

[modify] https://crrev.com/01d3ae00d277726b8aaee572120a5ba2a31d511e/pnacl/COMPONENT_REVISIONS

Comment 2 by stich...@chromium.org, Jan 12 2017

Project Member
Owner: stich...@chromium.org
Status: Fixed (was: New)

Comment 3 by stich...@chromium.org, Jan 12 2017

Project Member
Components: -SDK Subzero PNaCl

Comment 4 by bugdroid1@chromium.org, May 3 2017

Project Member
The following revision refers to this bug:
  https://swiftshader.googlesource.com/SwiftShader.git/+/373913fa0a4c7c4dd23ada9e6b4e1780fdea0096

commit 373913fa0a4c7c4dd23ada9e6b4e1780fdea0096
Author: Jim Stichnoth <stichnot@chromium.org>
Date: Fri Dec 23 14:07:07 2016

Subzero: Legalize the movzx argument.

The movzx operand must be a register or memory operand.  An immediate operand is not allowed.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4384
R=jpp@chromium.org

Review URL: https://codereview.chromium.org/2598153002 .

[modify] https://crrev.com/373913fa0a4c7c4dd23ada9e6b4e1780fdea0096/src/IceTargetLoweringX86BaseImpl.h

Sign in to add a comment