llvm / llvm/llvm-project

[CIR] Lowering of no-proto aliases/functions not right?

Open
#213,024 1 comment 0 reactions 0 assignees View on GitHub
ClangIR
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

See this patch here: https://github.com/llvm/llvm-project/pull/212308

Particularly the attr-alias-no-proto.c test. We end up having a difference in the LLVM IR generated that we should probably fix. I suspect that our lowering of functions/aliases/calls isn't properly setting the 'variadic-ness' of these functions properly.

Contributor guide

Open the contributing guide

Research direction

Start with the attr-alias-no-proto.c test and compare the generated LLVM IR with the patch at https://github.com/llvm/llvm-project/pull/212308. Trace the lowering of the functions, aliases, and calls involved, focusing on whether their variadic status is set consistently. Done means the IR difference is resolved and the test passes with the expected output.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
compilers
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.