Error on building: "variable ‘elements’ set but not used" #162
Assignees
Labels
area-infrastructure
Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes.
Comments
|
This comment was originally written by [email protected] Added Area-Build, Triaged labels. |
|
Can you please try deleting your "out" directory, re-syncing via "gclient sync", and the building again? Then, please let us know if this repeats. Set owner to @dgrove. |
|
Added Accepted label. |
|
This comment was originally written by [email protected] Yes, this bug remains. |
|
This comment was originally written by [email protected] I do it from scratch and I get the same error. |
|
This comment was originally written by [email protected] I do it from scratch and I get the same error as well. Using Ubuntu 11.04. Install jdk and jre as suggested at dartwatch, but to no avail. |
|
This comment was originally written by [email protected] Getting the same error |
|
This comment was originally written by @chrisbu The quick fix (to get you building) is to edit the file third_party/v8/src/ia32/lithium-codegen-ia32.cc and comment out the three lines in error: line 2235, 3100 and 3101 which are mentioned in the error message above. |
|
This comment was originally written by [email protected] The 3.4 branch of v8 has been patched as of r9839. This issue should be gone (after another gclient sync). |
|
Closing - we haven't heard anything about this in a long time. Added Done label. |
|
Removed Area-Build label. |
DartBot
added
Type-Defect
area-infrastructure
This was referenced Oct 29, 2020
dart-bot
pushed a commit
that referenced
this issue
Jan 7, 2021
Changes: 63f1110 Make argument to featureValueOf dynamic (#164) 71eeaee Make the item argument to Match dynamic again (#163) 2071cfd Add types to more parameters (#162) 5d52720 Allow null-safe code to do `same(null)`. (#161) 05db35c Delete .test_config (#160) e8b4964 remove redundant experiment (#159) 48fe280 Bump SDK constraints for pub (#158) 225cc13 allow the 2.12 prerelease sdks (#157) b9ce574 Prepare for the 2.11 dev SDKs (#156) fe4e68a Update .travis.yml Change-Id: I90c8d9d2f21d6454f1ace977deb9efec0d29d93d Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/177881 Reviewed-by: David Morgan <[email protected]> Commit-Queue: Ivan Inozemtsev <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
Oct 12, 2021
This is a reland of: https://dart-review.googlesource.com/c/sdk/+/213349 Reverted in https://dart-review.googlesource.com/c/sdk/+/213440 due to http://b/199988795 The two failures in that issue has been addressed by "ac53249 Fix timeout" and "117d3a1 Allow sendRaw to send Map<String, dynamic>" Most of these commits stem from a non-squashed merge. Changes: ``` > git log --format="%C(auto) %h %s" e0780cd..ac53249 https://dart.googlesource.com/usage.git/+/ac53249 Fix timeout (#162) https://dart.googlesource.com/usage.git/+/117d3a1 Allow sendRaw to send Map<String, dynamic> (#161) https://dart.googlesource.com/usage.git/+/016dd30 Prepare release (#160) https://dart.googlesource.com/usage.git/+/076c653 Merge pull request #152 from sigurdm/batching https://dart.googlesource.com/usage.git/+/1d6b343 Remove debug print https://dart.googlesource.com/usage.git/+/c073fb7 Use correct limit for bytes https://dart.googlesource.com/usage.git/+/1d89875 Send batch early if full https://dart.googlesource.com/usage.git/+/2896a85 Update README.md https://dart.googlesource.com/usage.git/+/da20fab Delete travis.sh https://dart.googlesource.com/usage.git/+/7c351a3 Merge https://dart.googlesource.com/usage.git/+/0424091 Update build.yaml (#159) https://dart.googlesource.com/usage.git/+/4f2a050 Use a Duration for batching delay https://dart.googlesource.com/usage.git/+/aa3f3ce address review https://dart.googlesource.com/usage.git/+/318ebe5 Fix https://dart.googlesource.com/usage.git/+/71ddec1 Send remaining hits on waitForLastPing https://dart.googlesource.com/usage.git/+/1c895a7 Parametrize client-id in test https://dart.googlesource.com/usage.git/+/0ae7733 Fix pubspec https://dart.googlesource.com/usage.git/+/2255b3d Merge remote-tracking branch 'origin/master' into batching https://dart.googlesource.com/usage.git/+/14d3702 Merge remote-tracking branch 'origin/master' into batching https://dart.googlesource.com/usage.git/+/1c3393d Handle batching implicit waiting for a 'batchDelay' future https://dart.googlesource.com/usage.git/+/219b764 TODO about size of payloads https://dart.googlesource.com/usage.git/+/aaee6d9 Fix AnalyticsMock https://dart.googlesource.com/usage.git/+/a8ec5b4 Remove mockito as dep - didn't use it https://dart.googlesource.com/usage.git/+/e26fcce Allow batching of hits ``` Diff: https://dart.googlesource.com/usage.git/+/e0780cd8b2f8af69a28dc52678ffe8492da27d06~..ac53249e5230a77624f46c07c2ed965efcfc0c59/ Change-Id: I1284db2a23a3e9ac037d1d2a80627c0caa1e9385 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/216184 Reviewed-by: Jonas Jensen <[email protected]> Commit-Queue: Sigurd Meldgaard <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
Oct 14, 2021
This reverts commit b2ea872. Reason for revert: Broke //third_party/fuchsia_gui_tools/fuchsia_devtools:tests_test_fuchsia_devtools_app_test.host Original change's description: > Bump usage to ac53249e5230a77624f46c07c2ed965efcfc0c59 > > This is a reland of: https://dart-review.googlesource.com/c/sdk/+/213349 > Reverted in https://dart-review.googlesource.com/c/sdk/+/213440 due to http://b/199988795 > > The two failures in that issue has been addressed by "ac53249 Fix timeout" and "117d3a1 Allow sendRaw to send Map<String, dynamic>" > > > Most of these commits stem from a non-squashed merge. > > Changes: > ``` > > git log --format="%C(auto) %h %s" e0780cd..ac53249 > https://dart.googlesource.com/usage.git/+/ac53249 Fix timeout (#162) > https://dart.googlesource.com/usage.git/+/117d3a1 Allow sendRaw to send Map<String, dynamic> (#161) > https://dart.googlesource.com/usage.git/+/016dd30 Prepare release (#160) > https://dart.googlesource.com/usage.git/+/076c653 Merge pull request #152 from sigurdm/batching > https://dart.googlesource.com/usage.git/+/1d6b343 Remove debug print > https://dart.googlesource.com/usage.git/+/c073fb7 Use correct limit for bytes > https://dart.googlesource.com/usage.git/+/1d89875 Send batch early if full > https://dart.googlesource.com/usage.git/+/2896a85 Update README.md > https://dart.googlesource.com/usage.git/+/da20fab Delete travis.sh > https://dart.googlesource.com/usage.git/+/7c351a3 Merge > https://dart.googlesource.com/usage.git/+/0424091 Update build.yaml (#159) > https://dart.googlesource.com/usage.git/+/4f2a050 Use a Duration for batching delay > https://dart.googlesource.com/usage.git/+/aa3f3ce address review > https://dart.googlesource.com/usage.git/+/318ebe5 Fix > https://dart.googlesource.com/usage.git/+/71ddec1 Send remaining hits on waitForLastPing > https://dart.googlesource.com/usage.git/+/1c895a7 Parametrize client-id in test > https://dart.googlesource.com/usage.git/+/0ae7733 Fix pubspec > https://dart.googlesource.com/usage.git/+/2255b3d Merge remote-tracking branch 'origin/master' into batching > https://dart.googlesource.com/usage.git/+/14d3702 Merge remote-tracking branch 'origin/master' into batching > https://dart.googlesource.com/usage.git/+/1c3393d Handle batching implicit waiting for a 'batchDelay' future > https://dart.googlesource.com/usage.git/+/219b764 TODO about size of payloads > https://dart.googlesource.com/usage.git/+/aaee6d9 Fix AnalyticsMock > https://dart.googlesource.com/usage.git/+/a8ec5b4 Remove mockito as dep - didn't use it > https://dart.googlesource.com/usage.git/+/e26fcce Allow batching of hits > > ``` > > Diff: https://dart.googlesource.com/usage.git/+/e0780cd8b2f8af69a28dc52678ffe8492da27d06~..ac53249e5230a77624f46c07c2ed965efcfc0c59/ > Change-Id: I1284db2a23a3e9ac037d1d2a80627c0caa1e9385 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/216184 > Reviewed-by: Jonas Jensen <[email protected]> > Commit-Queue: Sigurd Meldgaard <[email protected]> # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: Ib58c23769f263435237881e6a57a4cab18bc2850 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/216780 Reviewed-by: Sigurd Meldgaard <[email protected]> Reviewed-by: Jonas Jensen <[email protected]> Commit-Queue: Sigurd Meldgaard <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
Oct 14, 2021
This is a reland of https://dart-review.googlesource.com/c/sdk/+/216184 that was reverted due to b/203034967 The new commit f0cb8f7 `Don't batch per default` works around the issue (manually tested internally). Changes: ``` > git log --format="%C(auto) %h %s" e0780cd..f0cb8f7 https://dart.googlesource.com/usage.git/+/f0cb8f7 Don't batch per default (#164) https://dart.googlesource.com/usage.git/+/ac53249 Fix timeout (#162) https://dart.googlesource.com/usage.git/+/117d3a1 Allow sendRaw to send Map<String, dynamic> (#161) https://dart.googlesource.com/usage.git/+/016dd30 Prepare release (#160) https://dart.googlesource.com/usage.git/+/076c653 Merge pull request #152 from sigurdm/batching https://dart.googlesource.com/usage.git/+/1d6b343 Remove debug print https://dart.googlesource.com/usage.git/+/c073fb7 Use correct limit for bytes https://dart.googlesource.com/usage.git/+/1d89875 Send batch early if full https://dart.googlesource.com/usage.git/+/2896a85 Update README.md https://dart.googlesource.com/usage.git/+/da20fab Delete travis.sh https://dart.googlesource.com/usage.git/+/7c351a3 Merge https://dart.googlesource.com/usage.git/+/0424091 Update build.yaml (#159) https://dart.googlesource.com/usage.git/+/4f2a050 Use a Duration for batching delay https://dart.googlesource.com/usage.git/+/aa3f3ce address review https://dart.googlesource.com/usage.git/+/318ebe5 Fix https://dart.googlesource.com/usage.git/+/71ddec1 Send remaining hits on waitForLastPing https://dart.googlesource.com/usage.git/+/1c895a7 Parametrize client-id in test https://dart.googlesource.com/usage.git/+/0ae7733 Fix pubspec https://dart.googlesource.com/usage.git/+/2255b3d Merge remote-tracking branch 'origin/master' into batching https://dart.googlesource.com/usage.git/+/14d3702 Merge remote-tracking branch 'origin/master' into batching https://dart.googlesource.com/usage.git/+/1c3393d Handle batching implicit waiting for a 'batchDelay' future https://dart.googlesource.com/usage.git/+/219b764 TODO about size of payloads https://dart.googlesource.com/usage.git/+/aaee6d9 Fix AnalyticsMock https://dart.googlesource.com/usage.git/+/a8ec5b4 Remove mockito as dep - didn't use it https://dart.googlesource.com/usage.git/+/e26fcce Allow batching of hits ``` Diff: https://dart.googlesource.com/usage.git/+/e0780cd8b2f8af69a28dc52678ffe8492da27d06~..f0cb8f7cce8b675255c81488dbab8cf9f2f56404/ Change-Id: Iccac84c5182953209d7c05ac47cbde82e59a590d Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/216803 Commit-Queue: Sigurd Meldgaard <[email protected]> Reviewed-by: Jonas Jensen <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
Sep 5, 2023
…ion, dartdoc, ecosystem, file, glob, html, http_multi_server, http_parser, json_rpc_2, mockito, package_config, protobuf, pub_semver, source_maps, source_span, sync_http, test_reflective_loader, usage, vector_math, web_socket_channel, webdriver Revisions updated by `dart tools/rev_sdk_deps.dart`. args (https://github.com/dart-lang/args/compare/da56b18..5a4e16f): 5a4e16f 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#254) bazel_worker (https://github.com/dart-lang/bazel_worker/compare/f950bbf..159e671): 159e671 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#80) benchmark_harness (https://github.com/dart-lang/benchmark_harness/compare/fde73cb..7d0d28e): 7d0d28e 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#92) characters (https://github.com/dart-lang/characters/compare/ec844db..7633a16): 7633a16 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#86) collection (https://github.com/dart-lang/collection/compare/1a9b7eb..91afde4): 91afde4 2023-09-02 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#311) dartdoc (https://github.com/dart-lang/dartdoc/compare/695b218..a32ba3a): a32ba3a1 2023-09-05 Parker Lougheed Enable accidentally disabled reflective parameter test (#3490) 0c0cb4ed 2023-09-04 dependabot[bot] Bump actions/checkout from 3.5.3 to 4.0.0 (#3492) 08d8d9c4 2023-09-01 Sam Rawlins Implement much support for extension types (#3489) ecosystem (https://github.com/dart-lang/ecosystem/compare/89e58de..2e6c3ec): 2e6c3ec 2023-09-05 Hossein Yousefi Fix Publish Workflow (#162) 2e532e3 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#161) file (https://github.com/google/file.dart/compare/5d9a602..a18ad1c): a18ad1c 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.2 to 3.6.0 (#228) glob (https://github.com/dart-lang/glob/compare/5b24393..9c1996f): 9c1996f 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#82) html (https://github.com/dart-lang/html/compare/4060496..a1b193e): a1b193e 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#225) http_multi_server (https://github.com/dart-lang/http_multi_server/compare/aa128cf..9d62ea3): 9d62ea3 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#57) http_parser (https://github.com/dart-lang/http_parser/compare/c14fbf6..d2d03e7): d2d03e7 2023-09-02 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#77) json_rpc_2 (https://github.com/dart-lang/json_rpc_2/compare/509f71e..50a3786): 50a3786 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#101) mockito (https://github.com/dart-lang/mockito/compare/f5abf11..412c0be): 412c0be 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#692) package_config (https://github.com/dart-lang/package_config/compare/981c49d..ae7ad83): ae7ad83 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#140) protobuf (https://github.com/dart-lang/protobuf/compare/5e8f36b..c16bc89): c16bc89 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.2 to 3.6.0 (#873) pub_semver (https://github.com/dart-lang/pub_semver/compare/028b435..f0be74a): f0be74a 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#91) source_maps (https://github.com/dart-lang/source_maps/compare/97c4833..eb3d40a): eb3d40a 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#81) source_span (https://github.com/dart-lang/source_span/compare/37735ae..48d0f57): 48d0f57 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#101) sync_http (https://github.com/dart-lang/sync_http/compare/c3d6ad4..8233f74): 8233f74 2023-09-01 Devon Carew Merge pull request #37 from google/dependabot/github_actions/actions/checkout-3.6.0 57dc02b 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.2 to 3.6.0 test_reflective_loader (https://github.com/dart-lang/test_reflective_loader/compare/0bfaad9..45c57d6): 45c57d6 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#51) usage (https://github.com/dart-lang/usage/compare/09bb847..7b12d51): 7b12d51 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#197) vector_math (https://github.com/google/vector_math.dart/compare/88bada3..d54af8a): d54af8a 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.2 to 3.6.0 (#301) web_socket_channel (https://github.com/dart-lang/web_socket_channel/compare/4d1b543..af945f1): af945f1 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.3 to 3.6.0 (#283) webdriver (https://github.com/google/webdriver.dart/compare/20ec47f..21976d6): 21976d6 2023-09-01 dependabot[bot] Bump actions/checkout from 3.5.2 to 3.6.0 (#279) 352b9b6 2023-09-01 dependabot[bot] Bump nanasess/setup-chromedriver from 1.1.0 to 2.2.0 (#278) Change-Id: I0b1f7a8851a7ee992d5b67a221ab523f55b51240 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/324261 Auto-Submit: Devon Carew <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]>
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue was originally filed by [email protected]
What steps will reproduce the problem?
3.
What is the expected output? What do you see instead?
popsul@oo-pc:
/depot_tools/dart/dart$ gclient sync/depot_tools/dart/dart$ LANG=en_US.UTF-8 ./tools/build.py --arch=ia32 -v1>________ running 'svn update /home/popsul/depot_tools/dart/all.deps --non-interactive --accept postpone --ignore-externals' in '/home/popsul/depot_tools/dart'
1>At revision 507.
1>Syncing projects: 0% ( 0/ 3)
2>________ running 'svn update /home/popsul/depot_tools/dart/dart --non-interactive --accept postpone --ignore-externals' in '/home/popsul/depot_tools/dart'
2>At revision 507.
8>________ running 'svn update /home/popsul/depot_tools/dart/dart/third_party/v8 --non-interactive --accept postpone --ignore-externals' in '/home/popsul/depot_tools/dart'
8>At revision 9675.
Syncing projects: 88% (8/9), done./third_party/v8
popsul@oo-pc:
make -j 2 BUILDTYPE=Debug_ia32 V=1 all
g++ '-DENABLE_DEBUGGER_SUPPORT' '-DV8_TARGET_ARCH_IA32' -Ithird_party/v8/src -Ithird_party/v8/src/ia32 -Werror -Wall -Wextra -Wno-unused-parameter -Wnon-virtual-dtor -g -ansi -fno-rtti -fno-exceptions -fPIC -fvisibility=hidden -fvisibility-inlines-hidden -m32 -O0 -MMD -MF out/Debug_ia32/.deps/out/Debug_ia32/obj.host/v8_base/third_party/v8/src/ia32/lithium-codegen-ia32.o.d.raw -c -o out/Debug_ia32/obj.host/v8_base/third_party/v8/src/ia32/lithium-codegen-ia32.o third_party/v8/src/ia32/lithium-codegen-ia32.cc
g++ '-DENABLE_DEBUGGER_SUPPORT' '-DV8_TARGET_ARCH_IA32' -Ithird_party/v8/src -Ithird_party/v8/src/ia32 -Werror -Wall -Wextra -Wno-unused-parameter -Wnon-virtual-dtor -g -ansi -fno-rtti -fno-exceptions -fPIC -fvisibility=hidden -fvisibility-inlines-hidden -m32 -O0 -MMD -MF out/Debug_ia32/.deps/out/Debug_ia32/obj.host/v8_base/third_party/v8/src/ia32/macro-assembler-ia32.o.d.raw -c -o out/Debug_ia32/obj.host/v8_base/third_party/v8/src/ia32/macro-assembler-ia32.o third_party/v8/src/ia32/macro-assembler-ia32.cc
third_party/v8/src/ia32/lithium-codegen-ia32.cc: In member function ‘void v8::internal::LCodeGen::DoLoadKeyedFastDoubleElement(v8::internal::LLoadKeyedFastDoubleElement*)’:
third_party/v8/src/ia32/lithium-codegen-ia32.cc:2235:12: error: variable ‘elements’ set but not used [-Werror=unused-but-set-variable]
third_party/v8/src/ia32/lithium-codegen-ia32.cc: In member function ‘void v8::internal::LCodeGen::DoStoreKeyedFastDoubleElement(v8::internal::LStoreKeyedFastDoubleElement*)’:
third_party/v8/src/ia32/lithium-codegen-ia32.cc:3100:12: error: variable ‘elements’ set but not used [-Werror=unused-but-set-variable]
third_party/v8/src/ia32/lithium-codegen-ia32.cc:3101:12: error: variable ‘key’ set but not used [-Werror=unused-but-set-variable]
cc1plus: all warnings being treated as errors
make: *** [out/Debug_ia32/obj.host/v8_base/third_party/v8/src/ia32/lithium-codegen-ia32.o] Error 1
make: *** Waiting for unfinished jobs....
BUILD FAILED
What version of the product are you using? On what operating system?
Ubuntu 11.10, gcc 4.6.1
The text was updated successfully, but these errors were encountered: