[llvm] [LoopUtils] Saturate at INT_MAX when estimating TC (PR #129683)

[Florian Hahn via llvm-commits]

================
@@ -318,9 +318,8 @@ void addStringMetadataToLoop(Loop *TheLoop, const char *MDString,
 /// Returns a loop's estimated trip count based on branch weight metadata.
 /// In addition if \p EstimatedLoopInvocationWeight is not null it is
 /// initialized with weight of loop's latch leading to the exit.
-/// Returns a valid positive trip count, or std::nullopt when a meaningful
-/// estimate cannot be made (including when the trip count wouldn't fit in the
-/// result type).
+/// Returns a valid positive trip count, saturated at UINT_MAX, or std::nullopt
----------------
fhahn wrote:

Might be better to clarify that this doesn't aways return a valid trip count, but a lower bound on the trip count? 


https://github.com/llvm/llvm-project/pull/129683