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

ICE while generating debug information #51926

Closed
mordante opened this issue Nov 22, 2021 · 5 comments
Closed

ICE while generating debug information #51926

mordante opened this issue Nov 22, 2021 · 5 comments
Labels
bugzilla Issues migrated from bugzilla debuginfo

Comments

@mordante
Copy link
Member

Bugzilla Link 52584
Version trunk
OS Linux
Attachments backtrace of the crash, Reduced source, Reduced run script
CC @dwblaikie,@JDevlieghere,@walkerkd,@pogo59

Extended Description

While testing https://reviews.llvm.org/D70631 with the libc++ buildkite CI this build results in an crash https://buildkite.com/llvm-project/libcxx-ci/builds/6587#d7ec64f2-e31a-4d73-8d6c-e118e63a0cc1

The CI job builds clang and uses that Clang to build the patch.
https://github.com/llvm/llvm-project/blob/main/libcxx/utils/ci/run-buildbot#L470
Two work-arounds are

  • Remove -DLLVM_ENABLE_ASSERTIONS=ON
  • Change the build type to Release

Attached the backtrace and reduced reproducer.
@dwblaikie
Copy link
Collaborator

Seems to reproduce with any debug info generation, not gmlt (no variables), but otherwise it'll crash.

@dwblaikie
Copy link
Collaborator

Specifically the crash is around:
clang-tot: /usr/local/google/home/blaikie/dev/llvm/src/llvm/lib/CodeGen/AsmPrinter/DwarfExpression.cpp:509: void llvm::DwarfExpression::addExpression(llvm::DIExpressionCursor &&, llvm::function_ref<bool (unsigned int, DIExpressionCursor &)>): Assertion `SizeInBits >= OffsetInBits - FragmentOffset && "size underflow"' failed.

@dwblaikie
Copy link
Collaborator

This is probably a dup of #51675 , or a similar class of issues.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
@rickyz
Copy link
Member

rickyz commented Dec 11, 2021

This should be fixed by c7c84b9

PeiMu pushed a commit to PeiMu/llvm-project that referenced this issue Dec 13, 2021
@feg208 @tstellar
fixes bug llvm#51926 where dangling comma caused overrun
ec4829e 
bug 51926 identified an issue where a dangling comma caused the cell count to be to off by one

Differential Revision: https://reviews.llvm.org/D110481

(cherry picked from commit a36227c)
@mordante
Copy link
Member Author

I landed https://reviews.llvm.org/D70631 and there are no more issues so this bug has been fixed.

This was referenced Dec 17, 2021
Closed
Open
josemonsalve2 pushed a commit to josemonsalve2/llvm-project that referenced this issue Jun 15, 2022
@feg208
fixes bug llvm#51926 where dangling comma caused overrun
a36227c 
bug 51926 identified an issue where a dangling comma caused the cell count to be to off by one

Differential Revision: https://reviews.llvm.org/D110481
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla debuginfo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rickyz @mordante @dwblaikie