Skip to content

Apply C# infrastructure updates from PR #397#6

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/bring-pr-up-to-date
Draft

Apply C# infrastructure updates from PR #397#6
Copilot wants to merge 2 commits intomainfrom
copilot/bring-pr-up-to-date

Conversation

Copy link

Copilot AI commented Jan 22, 2026

Resurrects changes from 2023 PR hanabi1224#397 to enable .NET preview and NativeAOT benchmark variants.

Infrastructure

  • Added clang to .github/dotnet.sh - required for NativeAOT linker
  • Documented unzip dependency in README

CI/CD

  • Enabled csharp_preview and csharp_native_aot in workflow matrix
  • Allows benchmarking .NET 10 preview features and native compilation

Benchmark Configs

  • Added multiprocessor regex-redux implementations (5-m.cs, 6-m.cs) across all C# configs
  • Fixed bench_csharp_native_aot.yaml lang identifier: csharpcsharp_native_aot
  • Enabled all scenarios in bench_csharp_preview.yaml (was disabled with most tests commented out)

Code Quality

  • Normalized procfsProcFs in 3 files for C# naming conventions
Original prompt

This section details on the original issue you should resolve

<issue_title>Bring PR up to date</issue_title>
<issue_description>Look at old PR hanabi1224#397 . rebase on main if necessary. Consider which changes there are worth keeping, and validate them. </issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add clang dependency to .github/dotnet.sh for NativeAOT support
- Add csharp_preview and csharp_native_aot to bench.yml workflow
- Add unzip requirement to README.md
- Add multiproc regex-redux files (5-m.cs, 6-m.cs) to bench configs
- Fix lang field in bench_csharp_native_aot.yaml
- Enable and uncomment scenarios in bench_csharp_preview.yaml
- Fix ProcFs naming convention (procfs → ProcFs) for consistency

Co-authored-by: danmoseley <6385855+danmoseley@users.noreply.github.com>
Copilot AI changed the title [WIP] Update PR with the latest changes from main Apply C# infrastructure updates from PR #397 Jan 22, 2026
Copilot AI requested a review from danmoseley January 22, 2026 16:37
@danmoseley
Copy link
Owner

@copilot did you change anything over the original PR?

Copy link
Author

Copilot AI commented Jan 22, 2026

@copilot did you change anything over the original PR?

No, the changes in commit 81bfdf2 are identical to the original PR hanabi1224#397. All 9 files modified match exactly with the original PR's changes.

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.

Bring PR up to date

2 participants