[ARROW-9118] [C++] Add more general BoundsCheck function that also checks for arbitrary lower limits in integer arrays

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Component/s: C++
• External issue URL: https://github.com/apache/arrow/issues/25230
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 12/Jun/20 18:41
• Updated: 11/Jan/23 08:04
• Resolved: 22/Jun/20 23:59

Description

See ARROW-9083. The current IndexBoundsCheck is specialized to skip a comparison for unsigned integers and uses 0 as the lower bound for signed integers. This could be generalized so that we could check e.g. if int64 values will fit in the int32 range

Issue Links

• relates to: ARROW-9083 [R] collect int64, uint32, uint64 as R integer type if not out of bounds
• relates to: ARROW-9197 [C++] Revamp numeric casts: faster performance and reduced binary size