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

clang-format crashes on clang/test/Sema/attr-external-source-symbol.c #54368

Closed
owenca opened this issue Mar 14, 2022 · 2 comments
Closed

clang-format crashes on clang/test/Sema/attr-external-source-symbol.c #54368

owenca opened this issue Mar 14, 2022 · 2 comments
Assignees
Labels
clang-format crash Prefer [crash-on-valid] or [crash-on-invalid]

Comments

@owenca
Copy link
Contributor

owenca commented Mar 14, 2022

The minimum test case to reproduce the crash using the latest trunk version:

$ ls .clang-format
ls: .clang-format: No such file or directory
$ clang-format
int (^)() = ^ ()
  external_source_symbol() { //
  return 1;
};
Assertion failed: ((!TypeIsFinalized || T == Type) && "Please use overwriteFixedType to change a fixed type."), function setType, file FormatToken.h, line 351.
...

May be a duplicate of #54367.

@owenca owenca added clang-format crash Prefer [crash-on-valid] or [crash-on-invalid] labels Mar 14, 2022
@llvmbot
Copy link
Collaborator

llvmbot commented Mar 14, 2022

@llvm/issue-subscribers-clang-format

@HazardyKnusperkeks HazardyKnusperkeks self-assigned this Mar 14, 2022
@HazardyKnusperkeks HazardyKnusperkeks added the awaiting-review Has pending Phabricator review label Mar 14, 2022
@HazardyKnusperkeks
Copy link
Contributor

Review https://reviews.llvm.org/D121596

@github-actions github-actions bot removed the awaiting-review Has pending Phabricator review label Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang-format crash Prefer [crash-on-valid] or [crash-on-invalid]
Projects
None yet
Development

No branches or pull requests

3 participants