Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[chrome.v8] fixed the error libs not defined #2

Merged

Conversation

unicornx
Copy link

when gn with target_os = "android" and target_cpu = "riscv64"

libs may be not defined when goto

  if (v8_current_cpu == "riscv64" || v8_current_cpu == "riscv32") {
    libs += [ "atomic" ]
  }

when gn with target_os = "android" and target_cpu = "riscv64"

libs may be not defined when goto

```
  if (v8_current_cpu == "riscv64" || v8_current_cpu == "riscv32") {
    libs += [ "atomic" ]
  }
```
@unicornx unicornx self-assigned this Feb 19, 2023
@unicornx unicornx merged commit aef3aed into aosp-riscv:riscv64_109.0.5414.87_dev Feb 19, 2023
@unicornx unicornx changed the title fixed the error libs not defined [v8] fixed the error libs not defined Feb 28, 2023
@unicornx unicornx changed the title [v8] fixed the error libs not defined [chrome.v8] fixed the error libs not defined Feb 28, 2023
unicornx pushed a commit that referenced this pull request May 22, 2023
... on non-ChromeOS devices.

Bug: v8:11880, chromium:1400973
Change-Id: Id9914572d44c07d0ae1a1bd5d1cb25e8270c864b
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4287218
Auto-Submit: Igor Sheludko <[email protected]>
Reviewed-by: Toon Verwaest <[email protected]>
Commit-Queue: Toon Verwaest <[email protected]>
Cr-Commit-Position: refs/heads/main@{#86129}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant