-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Enable client-side accessor generation for strongly typed enum attributes #12731
Enable client-side accessor generation for strongly typed enum attributes #12731
Conversation
PR #12731: Size comparison from a7dd25e to 9ea2ac0 Increases above 0.2%:
Increases (1 build for linux)
Full report (15 builds for efr32, k32w, linux, p6, qpg, telink)
|
9ea2ac0
to
770fbf1
Compare
PR #12731: Size comparison from 1f4024a to 770fbf1 Increases above 0.2%:
Increases (1 build for linux)
Full report (12 builds for k32w, linux, p6, qpg, telink)
|
770fbf1
to
b73c868
Compare
PR #12731: Size comparison from 1f4024a to b73c868 Increases above 0.2%:
Increases (1 build for linux)
Full report (15 builds for efr32, k32w, linux, p6, qpg, telink)
|
c70e51a
to
82e1da2
Compare
PR #12731: Size comparison from b1d7073 to 82e1da2 Increases above 0.2%:
Increases (1 build for linux)
Full report (12 builds for efr32, linux, p6, qpg, telink)
|
/rebase |
82e1da2
to
c02618b
Compare
PR #12731: Size comparison from e4f3246 to c02618b Increases above 0.2%:
Increases (1 build for linux)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
fast track: change created and approved by domain owners |
Problem
Attributes whose types are named enums did not have client-side codegen working right, so had it disabled.
Change overview
Enable it.
Testing
Manually verified that we are now generating the relevant accessor code on the client side.