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

[SOL][BPF] Enable Solana BPF extensions as subtarget feature #9

Merged
merged 1 commit into from
Apr 12, 2021
Merged

[SOL][BPF] Enable Solana BPF extensions as subtarget feature #9

merged 1 commit into from
Apr 12, 2021

Conversation

dmakarov
Copy link
Collaborator

No description provided.

@dmakarov dmakarov force-pushed the solana-rustc/11.0-2021-01-05 branch from f0c6426 to 8b8d7a1 Compare April 11, 2021 10:30
@dmakarov dmakarov force-pushed the solana-rustc/11.0-2021-01-05 branch from 8b8d7a1 to 59a5a45 Compare April 11, 2021 10:51
@dmakarov dmakarov merged commit 91de63f into anza-xyz:solana-rustc/11.0-2021-01-05 Apr 12, 2021
dmakarov added a commit to dmakarov/llvm-project that referenced this pull request May 16, 2021
@dmakarov
[SOL][BPF] Enable Solana BPF extensions as subtarget feature (anza-xyz#9
9943f50 
)
@OluAgunloye OluAgunloye mentioned this pull request Sep 17, 2021
Closed
dmakarov pushed a commit to dmakarov/llvm-project that referenced this pull request Mar 17, 2023
@yonghong-song @tstellar
[clang][Sema] Fix a clang crash with btf_type_tag
80a9fc8 
For the following program,
  $ cat t.c
  struct t {
   int (__attribute__((btf_type_tag("rcu"))) *f)();
   int a;
  };
  int foo(struct t *arg) {
    return arg->a;
  }
Compiling with 'clang -g -O2 -S t.c' will cause a failure like below:
  clang: /home/yhs/work/llvm-project/clang/lib/Sema/SemaType.cpp:6391: void {anonymous}::DeclaratorLocFiller::VisitParenTypeLoc(clang::ParenTypeLoc):
         Assertion `Chunk.Kind == DeclaratorChunk::Paren' failed.
  PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script.
  Stack dump:
  ......
  anza-xyz#5 0x00007f89e4280ea5 abort (/lib64/libc.so.6+0x21ea5)
  anza-xyz#6 0x00007f89e4280d79 _nl_load_domain.cold.0 (/lib64/libc.so.6+0x21d79)
  anza-xyz#7 0x00007f89e42a6456 (/lib64/libc.so.6+0x47456)
  anza-xyz#8 0x00000000045c2596 GetTypeSourceInfoForDeclarator((anonymous namespace)::TypeProcessingState&, clang::QualType, clang::TypeSourceInfo*) SemaType.cpp:0:0
  anza-xyz#9 0x00000000045ccfa5 GetFullTypeForDeclarator((anonymous namespace)::TypeProcessingState&, clang::QualType, clang::TypeSourceInfo*) SemaType.cpp:0:0
  ......

The reason of the failure is due to the mismatch of TypeLoc and D.getTypeObject().Kind. For example,
the TypeLoc is
  BTFTagAttributedType 0x88614e0 'int  btf_type_tag(rcu)()' sugar
  |-ParenType 0x8861480 'int ()' sugar
  | `-FunctionNoProtoType 0x8861450 'int ()' cdecl
  |   `-BuiltinType 0x87fd500 'int'
while corresponding D.getTypeObject().Kind points to DeclaratorChunk::Paren, and
this will cause later assertion.

To fix the issue, similar to AttributedTypeLoc, let us skip BTFTagAttributedTypeLoc in
GetTypeSourceInfoForDeclarator().

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

Successfully merging this pull request may close these issues.
1 participant
@dmakarov