llvm / llvm/llvm-project

IR verifier does not enforce metadata structure of read_register/write_register operands

Open
#162,415 0 comments 0 reactions 0 assignees View on GitHub
accepts-invalid llvm:ir
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

This fix should not have been nececessary: https://github.com/llvm/llvm-project/pull/162414

The verifier accepts a raw MDString reference. That would be a better format, maybe we should auto upgrade to that

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the linked pull request and the IR verifier handling for read_register/write_register operands. Determine the required metadata structure and whether raw MDString references should be accepted or upgraded; done means the verifier rejects invalid operands and the intended compatibility behavior is covered.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.