[PATCH] D43079: [TTI CostModel] change default cost of FP ops to 1 (PR36280)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 09:49:33 PST 2018


spatel added inline comments.


================
Comment at: include/llvm/CodeGen/BasicTTIImpl.h:494
+    // assumption when they can provide more accurate information.
+    unsigned OpCost = 1;
 
----------------
ABataev wrote:
> Maybe it is better to add a virtual method that will return the default cost of the integer and floating point operations for the target? The default implementation should keep 1 and 2, but for X86 it should return 1 in both cases.
I don't know of any target where anything but '1' is the right default answer, so I'd rather not perpetuate this. I think it's better to correct the problems revealed by this change.


https://reviews.llvm.org/D43079





More information about the llvm-commits mailing list