M-Labs
/
nac3
8
0
Fork 5
Code Issues 87 Pull Requests 3 Releases Wiki Activity

Collapse all assertions into one fail block #391

New Issue
Closed
opened 2024-03-28 15:53:02 +08:00 by derppening · 1 comment
derppening commented 2024-03-28 15:53:02 +08:00
Collaborator

Currently, each ctx.make_assert call will generate a new fail block for that assertion. Investigate whether it is possible to collapse all these fail blocks into one for greatly reduced code size.

Currently, each `ctx.make_assert` call will generate a new fail block for that assertion. Investigate whether it is possible to collapse all these fail blocks into one for greatly reduced code size.
derppening commented 2024-04-02 17:49:06 +08:00
Poster
Collaborator

After looking into this, I think the better way forward is to reduce the number of assertion checks when the input is already sanitized, especially with regards to IRRT helper functions.

After looking into this, I think the better way forward is to reduce the number of assertion checks when the input is already sanitized, especially with regards to IRRT helper functions.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
enhance/issue-149-ndarray/numpy-func-take-ndarray
master
enhance/cargo-test-standalone-demos
issue-136
rpc-obj-as-param
no_init
iterators
escape-analysis
refactor_anto
Labels
Clear labels   
UROP0622
   
future-extension
   
high-priority
   
low-priority
No Label
UROP0622
future-extension
high-priority
low-priority
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/nac3#391
There is no content yet.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?