[PATCH] D95803: Ensure that InstructionCost actually implements a total ordering

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 07:05:03 PST 2021


sdesmalen added inline comments.


================
Comment at: llvm/include/llvm/Support/InstructionCost.h:39-40
 private:
   CostType Value;
   CostState State;
 
----------------
Would it be an idea to explicitly initialize these values with 0 and Valid, respectively?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95803



More information about the llvm-commits mailing list