ychenfo commented on pull request M-Labs/nac3#261
Optimize Kernel Invariant on Unwrap and Tupleupdated block names to `unwrap_none_unreachable` and `unwrap_none_exception`
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- 722e3df086 nac3core, artiq: optimize kernel invariant for tuple index
- ad9ad22cb8 nac3core: optimize unwrap KernelInvariant
- Compare 2 commits »
ychenfo commented on pull request M-Labs/nac3#261
Optimize Kernel Invariant on Unwrap and TupleOh yes, I think the `_bb` is redundant as we can easily tell whether a name is a lable to a basic block. I will remove these postfixes now.
ychenfo commented on pull request M-Labs/nac3#261
Optimize Kernel Invariant on Unwrap and Tuple> I guess it would make more sense to name the function get_tuple_element instead of get_tuple_index? force-pushed to update the name to `get_tuple_element`
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- c037724a34 nac3core, artiq: optimize kernel invariant for tuple index
ychenfo commented on issue M-Labs/nac3#185
add bound checks for list slice and slice assignmentShall we close this issue as this should be done with #255 and #260?
ychenfo commented on pull request M-Labs/nac3#261
Optimize Kernel Invariant on Unwrap and Tupleforce-pused to update the names of the basic blocks to be `unwrap_none_unreachable_bb` and `unwrap_none_exception_bb`
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- 3223724c72 nac3core, artiq: optimize kernel invariant for tuple index
- de1a729cef nac3core: optimize unwrap KernelInvariant
- Compare 2 commits »
ychenfo created pull request M-Labs/nac3#261
Optimize Kernel Invariant on Unwrap and Tupleychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- 47b5cb204f nac3core, artiq: optimize kernel invariant for tuple index
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- bc6ce3de0b nac3core, artiq: optimize kernel invariant for tuple index
- 8682f45725 nac3core: optimize unwrap KernelInvariant
- f66f66b3a4 nac3core, artiq: remove unnecessary ptr casts
- 6c485bc9dc nac3artiq: skip attribute writeback for option
- 089bba96a3 nac3artiq: get_obj_value take an additional argument for expected type
- Compare 6 commits »
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- 9e2ed6f40b nac3core: optimize unwrap KernelInvariant
ychenfo pushed to optimize_unwrap_kernelinvariant at M-Labs/nac3
- 8f9da668ff nac3core: move option unwrap to builtin.rs
ychenfo commented on pull request M-Labs/nac3#259
Get llvm value with expected type (#256)> It might be better to split the last commit into the symbol resolver change (get_obj_value now accepts an expected_ty param) and the refactoring (adding expected type to all to_basic_value_enum…
ychenfo pushed to get_symbol_val_with_type at M-Labs/nac3
- f66f66b3a4 nac3core, artiq: remove unnecessary ptr casts
- 6c485bc9dc nac3artiq: skip attribute writeback for option
- 089bba96a3 nac3artiq: get_obj_value take an additional argument for expected type
- 0e0871bc38 nac3core, artiq: to_basic_value_enum takes an argument indicating the expected type
- Compare 4 commits »
ychenfo pushed to get_symbol_val_with_type at M-Labs/nac3
- 73bcf7fc3f nac3core,artiq: remove unnecessary ptr casts
- d3cbe4dd14 nac3artiq: skip attribute writeback for option
- 6a81fafe8c nac3artiq: get_obj_value take an additional argument for expected type
- 788739f636 nac3core,artiq,standalone: to_basic_value_enum takes an argument indicating the expected type
- Compare 4 commits »
ychenfo commented on pull request M-Labs/nac3#259
Get llvm value with expected type (#256)force-pushed to update on current master branch
ychenfo pushed to get_symbol_val_with_type at M-Labs/nac3
- c7a37217ae nac3core,artiq: remove unnecessary ptr casts
- 6eb24aecfb nac3artiq: skip attribute writeback for option
- 2ac0026616 nac3core,artiq,standalone: symbol resolver getting symbol value with expected type
- 26187bff0b nac3core: add missing bound check and negative index handling for list subscription assignment
- Compare 4 commits »