[PATCH] D79483: [CostModel] Remove getInstructionCost.

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 00:29:40 PDT 2020

Previous message: [PATCH] D79483: [CostModel] Remove getInstructionCost.
Next message: [PATCH] D82935: [llvm-objcopy] Fix crash when removing symbol table at same time as adding a symbol
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

samparker added a comment.

> Is getUserCost a better name than getInstructionCost? I wonder if we're moving in the wrong direction.

I'd prefer to use getInstructionCost too.


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

https://reviews.llvm.org/D79483

Previous message: [PATCH] D79483: [CostModel] Remove getInstructionCost.
Next message: [PATCH] D82935: [llvm-objcopy] Fix crash when removing symbol table at same time as adding a symbol
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list