Fix bug in reshape_lazy calculation

[pfultz2]

Motivation

Fixes #4590.

Technical Details

Also added some additional unit tests.

Changelog Category

Add a CHANGELOG.md entry for any option other than Not Applicable

• • Added: New functionality.
• • Changed: Changes to existing functionality.
• • Removed: Functionality or support that has been removed. (Compared to a previous release)
• • Optimized: Component performance that has been optimized or improved.
• • Resolved Issues: Known issues from a previous version that have been resolved.
• • Not Applicable: This PR is not to be included in the changelog.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #4594   +/-   ##
========================================
  Coverage    92.20%   92.20%           
========================================
  Files          567      567           
  Lines        27880    27880           
========================================
  Hits         25706    25706           
  Misses        2174     2174           

Files with missing lines	Coverage Δ
src/include/migraphx/op/reshape_lazy.hpp	95.80% <100.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[CharlieL7]

Do we want to consolidate changes from my draft pr #4589?

[pfultz2]

Do we want to consolidate changes from my draft pr #4589?

It shouldn't be changed to a throw. It should only throw if its collapsing two non-contiguous dimensions.

[kahmed10]

LGTM, don't forget to mention this in the changelog