Re: [ot][spam][crazy] Doing Nothing / karl status

While I was building, I encountered an android compile crash and filed a bug for it with the android ndk. strangely, the crash occurred only on desktop, not on the mobile device, although I expect both should be using the same compiler. here's the bug i filed: https://github.com/android/ndk/issues/1645#issuecomment-1006995288 here's a copy-paste of its page: Skip to content Search or jump to… Pulls Issues Marketplace Explore @xloem android / ndk Public Code Issues 161 Pull requests Discussions Actions Projects 4 Wiki More [BUG] clang-12: error: clang frontend command failed with exit code 139 #1645 Open xloem opened this issue 2 days ago · 4 comments Comments @xloem xloem commented 2 days ago • 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 2 days ago @DanAlbert DanAlbert added this to Awaiting triage in LLVM via automation yesterday @DanAlbert DanAlbert added this to Triaged in r24 via automation yesterday @DanAlbert DanAlbert added this to Triaged in r23c via automation yesterday @DanAlbert DanAlbert assigned stephenhines and pirama-arumuga-nainar yesterday @pirama-arumuga-nainar Collaborator pirama-arumuga-nainar commented 15 hours ago 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 Collaborator pirama-arumuga-nainar commented 11 hours ago Fix is: commit 7654bb6303d290b19cad29137be810e69a0bf917 Author: Alexey Bataev 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 11 hours ago @pirama-arumuga-nainar Collaborator pirama-arumuga-nainar commented 11 hours ago 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 11 hours ago @pirama-arumuga-nainar pirama-arumuga-nainar moved this from Triaged to Needs prebuilt update in r23c 11 hours ago @pirama-arumuga-nainar pirama-arumuga-nainar moved this from Needs prebuilt update to Triaged in r23c 11 hours ago @DanAlbert Member DanAlbert commented 11 hours ago This need not be triaged for r24 as mentioned in the original report. Whoops, a little overzealous while batch triaging :) Thanks @xloem Leave a comment No file chosen Attach files by dragging & dropping, selecting or pasting them. Remember, contributions to this repository should follow its contributing guidelines. Assignees @stephenhines stephenhines @pirama-arumuga-nainar pirama-arumuga-nainar Labels bug Projects LLVM Awaiting prebuilt drop r23c Triaged Milestone No milestone Linked pull requests Successfully merging a pull request may close this issue. None yet Notifications Customize You’re receiving notifications because you authored the thread. 4 participants @xloem @DanAlbert @stephenhines @pirama-arumuga-nainar © 2022 GitHub, Inc. Terms Privacy Security Status Docs Contact GitHub Pricing API Training Blog About