KhronosGroup / KhronosGroup/SPIRV-LLVM-Translator

Do we need to pass the size of alloca for llvm.lifetime.start?

Open
#1,130 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
LLVM
Stars
625
Forks
279
Avg merge
3d 5h
Merged PRs (30d)
34

Description

Probably we do. And the same for `llvm.lifetime.end`. The concern is: what are consequences in code generated by device BE compilers?

_Originally posted by @AlexeySotkin in https://github.com/KhronosGroup/SPIRV-LLVM-Translator/pull/1119#r673892508_

Contributor guide

Open the contributing guide

Research direction

Start by reviewing pull request #1119 and the handling of llvm.lifetime.start and llvm.lifetime.end. Investigate whether the alloca size must be passed and what consequences this has for code generated by device BE compilers. Done means the required behavior and its impact are established.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.