[PATCH] D77319: [WIP] Remove the getNegatibleCost

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 2 10:17:40 PDT 2020


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:3534
                                        bool LegalOperations, bool ForCodeSize,
+                                       bool &IsProfitable,
                                        unsigned Depth = 0) const;
----------------
You might be better off keeping the NegatibleCost enum and using it as a threshold value instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77319





More information about the llvm-commits mailing list