[PATCH] D57445: [ARM] Fix TTI IntImmCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 06:33:46 PST 2019


samparker marked an inline comment as done.
samparker added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:111
   // Load from constantpool.
   return 3;
 }
----------------
xbolva00 wrote:
> Can we have a enum or so with these return constants and use them instead of “return 3”, “return 1”, ..
Sure, I'll use TargetCostConstants.


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

https://reviews.llvm.org/D57445





More information about the llvm-commits mailing list