[PATCH] D143942: [NFC] Replace -1U{LL} expressions with appropriate *_MAX macros in Support library.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 13 13:33:47 PST 2023


craig.topper added inline comments.


================
Comment at: llvm/include/llvm/Support/BlockFrequency.h:31
   /// Returns the maximum possible frequency, the saturation value.
-  static uint64_t getMaxFrequency() { return -1ULL; }
+  static uint64_t getMaxFrequency() { return ULLONG_MAX; }
 
----------------
Why not `UINT64_MAX`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143942



More information about the llvm-commits mailing list