[PATCH] D46760: [InstCombine] Enhance narrowUDivURem.

[Bixia Zheng via Phabricator via llvm-commits]

bixia added a comment.

What Sanjoy suggested doesn't (completely) solve the problem we have , where UDIV and UREM are used to calculate the dimensional indices from a linear index, and the dimensionally indices are then used to access tensors of different shapes (for example, two tensors being accessed both require broadcasting but along different dimensions). 
With the current state of https://reviews.llvm.org/D47113,  would it be acceptable if I revise my change here to only narrow i64 udiv/urem to i32 using the added patterns?


Repository:
  rL LLVM

https://reviews.llvm.org/D46760