[PATCH] D105108: [InstructionCost] Add saturation support.

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added a comment.

If hitting either one of the saturation boundaries was realistic enough to warrant this change, then why having both, large positive and large negative costs wouldn't be?  If cost1 > 0 and cost2 < 0, and they both exceed the bounds, then cost1+cost2 may be a finite number, reasonably close to 0, and yet completely meaningless.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105108/new/

https://reviews.llvm.org/D105108