Skip to content

[circt-verilog] bump Slang to 10.0#9667

Merged
TaoBi22 merged 1 commit intollvm:mainfrom
TaoBi22:slang-10
Feb 13, 2026
Merged

[circt-verilog] bump Slang to 10.0#9667
TaoBi22 merged 1 commit intollvm:mainfrom
TaoBi22:slang-10

Conversation

@TaoBi22
Copy link
Contributor

@TaoBi22 TaoBi22 commented Feb 13, 2026

Bumps to a later version of Slang (primarily so we can make use of MikePopoloski/slang@673c7a7 to reuse Slang's LRM clock inference on assertions)

Copy link
Contributor

@fabianschuiki fabianschuiki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment on lines -1748 to +1754
// CHECK: [[V0:%.+]] = moore.extract_ref %z3 from 2
// CHECK: [[V0_READ:%.+]] = moore.read [[V0]]
// CHECK: [[Z3_READ:%.+]] = moore.read %z3
// CHECK: [[C2_I32:%.+]] = moore.constant 2 : i32
// CHECK: [[V0_READ:%.+]] = moore.dyn_extract [[Z3_READ]] from [[C2_I32]]
// CHECK: [[C1:%.+]] = moore.extract_ref %w3 from 0
// CHECK: [[C0:%.+]] = moore.extract_ref %w3 from 1
// CHECK: [[C0_READ:%.+]] = moore.read [[C0]]
// CHECK: [[W3_READ:%.+]] = moore.read %w3
// CHECK: [[C1_I32:%.+]] = moore.constant 1 : i32
// CHECK: [[C0_READ:%.+]] = moore.dyn_extract [[W3_READ]] from [[C1_I32]]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's interesting how we suddenly produce a dynamic extract with a constant index, instead of just a plain old extract. @dinoruic ran into this in #9489 and #9547, too. No need to block this version bump, just a curious thing that we might want to track down a bit later.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I did think it was interesting that this changed (especially since it was only in one place - not that I'm going to complain about a smaller bump diff!)

@TaoBi22 TaoBi22 merged commit 703ce27 into llvm:main Feb 13, 2026
7 checks passed
unlsycn added a commit to unlsycn/circt that referenced this pull request Mar 2, 2026
This fixes a mismatch in CMakeLists.txt introduced by llvm#9667.

Signed-off-by: unlsycn <unlsycn@unlsycn.com>
unlsycn added a commit that referenced this pull request Mar 2, 2026
This fixes a mismatch in CMakeLists.txt introduced by #9667.

Signed-off-by: unlsycn <unlsycn@unlsycn.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants