[PATCH] D91174: [Analysis] Introduce a new InstructionCost class

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 09:30:48 PST 2020


nikic added inline comments.


================
Comment at: llvm/include/llvm/Analysis/InstructionCost.h:42
+  T Value;
+  CostState State;
+
----------------
Please make "invalid" a special value (like -1) instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91174



More information about the llvm-commits mailing list