New issue
Advanced search Search tips

Issue 593039 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue 589647



Sign in to add a comment

Regression: clang produces crashing protoc in 32-bit Linux builds

Project Member Reported by thakis@chromium.org, Mar 8 2016

Issue description

Use your favorite GYP_DEFINES but include target_arch=ia32:

$ GYP_DEFINES="disable_nacl=1 fastbuild=1 target_arch=ia32 component=shared_library use_goma=1" build/gyp_chromium
Updating projects from gyp files...


Then build any target that invokes protoc, e.g.

$ ninja -C out/Release/ -j 1000 ../../third_party/cros_system_api/dbus/mtp_file_entry.proto^
ninja: Entering directory `out/Release/'
[1/1] RULE Generating C++ and Python code from ../../third_party/cros_system_api/dbus/mtp_file_entry.proto
FAILED: cd ../../device/media_transfer_protocol; python ../../tools/protoc_wrapper/protoc_wrapper.py --include "" --protobuf "../../out/Release/gen/protoc_out/device/media_transfer_protocol/mtp_file_entry.pb.h" --proto-in-dir ../../third_party/cros_system_api/dbus --proto-in-file "mtp_file_entry.proto" "--use-system-protobuf=0" -- ../../out/Release/protoc --cpp_out ../../out/Release/gen/protoc_out/device/media_transfer_protocol --python_out ../../out/Release/pyproto/device/media_transfer_protocol
terminate called after throwing an instance of 'std::length_error'
  what():  vector::_M_range_insert
ninja: build stopped: subcommand failed.

 

Comment 1 by p...@chromium.org, Mar 8 2016

Owner: p...@chromium.org
Looking

Comment 2 by h...@chromium.org, Mar 8 2016

protoc invocation:

$ out/Release/protoc --cpp_out out/Release/gen/protoc_out/components/proximity_auth/cryptauth/proto --python_out out/Release/pyproto/components/proximity_auth/cryptauth/proto --proto_path=components/proximity_auth/cryptauth/proto components/proximity_auth/cryptauth/proto/securemessage.proto

Comment 3 by p...@chromium.org, Mar 8 2016

I'm having some trouble reproducing this (tried trunk chromium (r379889) with released clang as well as trunk (r262953) clang).

Comment 4 by p...@chromium.org, Mar 8 2016

Status: Fixed (was: Untriaged)
Looks like this is fixed upstream as of r262839 (confirmed by toggling the -enable-loop-load-elim flag).

Sign in to add a comment