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

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 29 06:40:48 PDT 2021


kparzysz added a comment.

Should the max/min values be treated as infinity?  With this implementation, (max+1)-1 = max-1.  Is that what we want?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105108



More information about the llvm-commits mailing list