[GitHub] [arrow-rs] jackwener opened a new issue #1427: Redundant `if` and `abs` in `shift()`

[GitBox <gi...@apache.org>]

jackwener opened a new issue #1427:
URL: https://github.com/apache/arrow-rs/issues/1427


   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   There are some redundant code for control flow
   
   **Describe the solution you'd like**
   Replace these code
   
   **Describe alternatives you've considered**
   `if` is control flow will break the cpu pipeline, we should reduce the useless code.
   abs() should remove, and replace with code in the `if`
   
   **Additional context**
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-rs] alamb closed issue #1427: Redundant `if` and `abs` in `shift()`

[GitBox <gi...@apache.org>]

alamb closed issue #1427:
URL: https://github.com/apache/arrow-rs/issues/1427


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org