[PATCH] D90597: [CostModel] Make target intrinsics cheap by default

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:1135
       return 0;
 
     if (ICA.isTypeBasedOnly())
----------------
Do you see any differences if we add the check here?
I've been staring at and stepping through this code for days, and I'm still not exactly sure how it works...
But it seems like if we can stub something out at this level, it's better than waiting for the type-based specialization.


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

https://reviews.llvm.org/D90597