[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

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

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

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

xloem opened this issue Jan 5, 2022 · 5 comments
Assignees
@stephenhines @pirama-arumuga-nainar
Labels
bug
Projects
LLVM r23c

Comments

@xloem
Copy link
xloem commented Jan 5, 2022 •
edited
Loading

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
@xloem xloem added the bug label Jan 5, 2022
@DanAlbert DanAlbert added this to Awaiting triage in LLVM via automation Jan 6, 2022
@DanAlbert DanAlbert added this to Triaged in r24 via automation Jan 6, 2022
@DanAlbert DanAlbert added this to Triaged in r23c via automation Jan 6, 2022
@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

@DanAlbert DanAlbert removed this from Triaged in r24 Jan 6, 2022
@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.

@pirama-arumuga-nainar pirama-arumuga-nainar moved this from Awaiting triage to Awaiting prebuilt drop in LLVM Jan 6, 2022
@pirama-arumuga-nainar pirama-arumuga-nainar moved this from Triaged to Needs prebuilt update in r23c Jan 6, 2022
@pirama-arumuga-nainar pirama-arumuga-nainar moved this from Needs prebuilt update to Triaged in r23c Jan 6, 2022
@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 DanAlbert moved this from Triaged to Needs prebuilt update in r23c Apr 21, 2022
@DanAlbert DanAlbert moved this from Needs prebuilt update to Needs cherry-pick in r23c Apr 22, 2022
@DanAlbert DanAlbert moved this from Awaiting prebuilt drop to Prebuilts submitted in LLVM Apr 22, 2022
@DanAlbert
Copy link
Member

Should be fixed in r23 build 8486889.

r23c automation moved this from Needs cherry-pick to Merged Apr 22, 2022
cyberknight777 pushed a commit to Neutron-Toolchains/aosp_llvm_android that referenced this issue May 30, 2022
@pirama-arumuga-nainar
[patches] Cherry pick CLS for: fix for openmp crash
bc441f3 
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
@DanAlbert
Update to LLVM r416183c2.
dec6c3f 
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
@DanAlbert
Changelog update for r23c toolchain.
f08c087 
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
Assignees

@stephenhines stephenhines

@pirama-arumuga-nainar pirama-arumuga-nainar

Labels
bug
Projects
LLVM
  
Prebuilts submitted
r23c
  
Merged
Milestone
No milestone
Development

No branches or pull requests
4 participants
@xloem @DanAlbert @stephenhines @pirama-arumuga-nainar