Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cgswords/loop value break #14030

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Cgswords/loop value break #14030

merged 1 commit into from
Oct 27, 2023

Conversation

cgswords
Copy link
Contributor

Description

This adds value expressions to loop breaks

Test Plan

Everything still works, plus new tests


If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.

Type of Change (Check all that apply)

  • protocol change
  • user-visible impact
  • breaking change for a client SDKs
  • breaking change for FNs (FN binary must upgrade)
  • breaking change for validators or node operators (must upgrade binaries)
  • breaking change for on-chain data layout
  • necessitate either a data wipe or data migration

Release notes

@vercel
Copy link

vercel bot commented Sep 30, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mysten-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 8:20pm
sui-typescript-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 8:20pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Visit Preview Oct 26, 2023 8:20pm
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Oct 26, 2023 8:20pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Oct 26, 2023 8:20pm

@cgswords cgswords force-pushed the cgswords/loop_value_break branch from ef6ebdd to 6f607ce Compare October 5, 2023 21:46
@vercel vercel bot temporarily deployed to Preview – mysten-ui October 5, 2023 21:47 Inactive
@cgswords cgswords force-pushed the cgswords/loop_value_break branch from 6f607ce to db86640 Compare October 5, 2023 22:35
@cgswords cgswords requested review from tzakian, tnowacki and oxade October 5, 2023 22:36
@vercel vercel bot temporarily deployed to Preview – mysten-ui October 5, 2023 22:36 Inactive
@cgswords cgswords changed the base branch from main to cgswords/revised_hlir_lowering October 5, 2023 22:37
@cgswords cgswords force-pushed the cgswords/loop_value_break branch from db86640 to f64e0f9 Compare October 5, 2023 23:03
@cgswords cgswords marked this pull request as ready for review October 5, 2023 23:03
@vercel vercel bot temporarily deployed to Preview – mysten-ui October 5, 2023 23:03 Inactive
@cgswords cgswords force-pushed the cgswords/loop_value_break branch from f64e0f9 to bd5dff9 Compare October 5, 2023 23:08
@vercel vercel bot temporarily deployed to Preview – mysten-ui October 5, 2023 23:09 Inactive
Copy link
Contributor

@tnowacki tnowacki left a comment

Choose a reason for hiding this comment

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

Overall looks good!

Some general thoughts:

  • It is fun that we just reuse Var but should we maybe make a new type, BlockLabel... or something?
  • See comments about possible simplifications to loc and join with () for while

@cgswords
Copy link
Contributor Author

It is fun that we just reuse Var but should we maybe make a new type, BlockLabel... or something?

I reused vars because those names should be treated hygienically during macro expansion in the future.

See comments about possible simplifications to loc and join with () for while

Done.

@cgswords cgswords force-pushed the cgswords/loop_value_break branch from b9ac3d9 to 83a7659 Compare October 26, 2023 20:20
@vercel vercel bot temporarily deployed to Preview – mysten-ui October 26, 2023 20:20 Inactive
@cgswords cgswords merged commit 4e86d98 into compiler-dev Oct 27, 2023
@cgswords cgswords deleted the cgswords/loop_value_break branch October 27, 2023 03:22
cgswords added a commit that referenced this pull request Oct 27, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Oct 27, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Oct 27, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Oct 27, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Oct 27, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 3, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 14, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 14, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 15, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 17, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 21, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 21, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 21, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
cgswords added a commit that referenced this pull request Nov 22, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
tzakian pushed a commit that referenced this pull request Nov 29, 2023
This adds value expressions to loop breaks

Everything still works, plus new tests

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration
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