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

[BUG] clang-12: error: clang frontend command failed with exit code 139 #1645

Closed
xloem opened this issue Jan 5, 2022 · 5 comments
Closed
Assignees
Labels

Comments

@xloem
Copy link

xloem commented Jan 5, 2022

Description

While building, the compiler crashed and informed me to report a bug. I have attached its generated data, which appears to likely form a test case.

clang.log
MaxUnpoolKernel_cpp_sh.tar.gz

Update: This crash went away for me when I switched to ndk 24-beta2.

Environment Details

  • NDK Version: 23.1.7779620
  • Build system: cmake ninja
  • Host OS: Linux Ubuntu 20.04.3, running from docker image termux/package-builder
  • ABI: arm64-v8a
  • NDK API level: 23
  • Device API level: n/a
@pirama-arumuga-nainar
Copy link
Collaborator

This need not be triaged for r24 as mentioned in the original report.

Crashes in r23 clang: SHA 8456c3a789285079ad35d146e487436b5a27b027
Fixed in r24 clang: SHA 0fe7531b75e54909a7cb7f7080634ab2053964ad

Next step is to identify a fix using git bisect and cherry-pick to the LTS compiler branch.

@pirama-arumuga-nainar
Copy link
Collaborator

Fix is:

commit 7654bb6303d290b19cad29137be810e69a0bf917
Author: Alexey Bataev <a.bataev@outlook.com>
Date:   Wed Mar 24 09:24:24 2021 -0700

    [OPENMP]Fix PR48571: critical/master in outlined contexts cause crash.
    
    If emit inlined region for master/critical directives, no need to clear
    lambda/block context data, otherwise the variables cannot be found and
    it causes a crash at compile time.
    
    Differential Revision: https://reviews.llvm.org/D99280

@pirama-arumuga-nainar
Copy link
Collaborator

https://android-review.googlesource.com/c/toolchain/llvm_android/+/1939337 cherry-picks it to the necessary branch. Just need a prebuilt update before r23c.

@DanAlbert
Copy link
Member

This need not be triaged for r24 as mentioned in the original report.

Whoops, a little overzealous while batch triaging :) Thanks

@DanAlbert
Copy link
Member

Should be fixed in r23 build 8486889.

cyberknight777 pushed a commit to Neutron-Toolchains/aosp_llvm_android that referenced this issue May 30, 2022
Bug: http://b/android/ndk#1645

7654bb6303d [OPENMP]Fix PR48571: critical/master in outlined contexts cause crash.

Test: N/A
Change-Id: If62f63f46ce22ba7d668ace5ef0dd7ab268171cf
MaoHan001 pushed a commit to riscv-android-src/platform-ndk that referenced this issue Jun 22, 2022
Changelog updates are in a separate commit to make cherry-picking to
master easier.

Bug: android/ndk#1590
Bug: android/ndk#1608
Bug: android/ndk#1619
Bug: android/ndk#1645
Bug: android/ndk#1672
Test: ./checkbuild.py && ./run_tests.py
Change-Id: Ie5571ed436cb0a3fe9ad675ed15f62fff4e978d6
(cherry picked from commit 59e8e507c2a2147c2bc806087c953dd36f6b1c41)
Merged-In: Ie5571ed436cb0a3fe9ad675ed15f62fff4e978d6
MaoHan001 pushed a commit to riscv-android-src/platform-ndk that referenced this issue Jun 22, 2022
Separate from the toolchain update to avoid merge conflicts.

Bug: android/ndk#1590
Bug: android/ndk#1608
Bug: android/ndk#1619
Bug: android/ndk#1645
Bug: android/ndk#1672
Test: None
Change-Id: I6e24e582dc0c300db173083009da9a1494360137
(cherry picked from commit 25ab62f84177b8f57782048a01a755c5730d6e6b)
Merged-In: I6e24e582dc0c300db173083009da9a1494360137
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants