llvm / llvm/circt

[Comb] ExtractOp lowBit Attribute vs. Operand.

Open
#3,287 0 comments 0 reactions 0 assignees View on GitHub
Comb
Dominant language
C++
Stars
2.2k
Forks
524
Avg merge
3d 2h
Merged PRs (30d)
46

Description

Currently, `comb.extract` has `lowBit` as an `Attribute`. Why is it not an operand?

I swear we've gone over this before, but I can't find the discussion anywhere.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the repository for the `comb.extract` operation and inspect how its `lowBit` attribute is defined and consumed. Review nearby operation definitions and any available history or discussion to determine whether changing it to an operand is intended; done requires a documented design decision and corresponding implementation scope.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
compilers
Issue type
Refactor
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.