[PATCH] D43769: [TTI] rename getArithmeticInstructionCost() to getUnitThroughput(); NFC
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 28 09:28:35 PST 2018
spatel added a comment.
At this point, I'll settle for just having an accurate function comment. :)
Let me update with just that much and see if we can reach consensus.
> Can the function be used for memory operations?
I don't think so - we have "getMemoryOpCost" for that purpose IIUC. So again, a renaming exercise would be best if we make the whole API uniform.
https://reviews.llvm.org/D43769
More information about the llvm-commits
mailing list