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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

I think this change should be fine for modern AArch64 chips. For example, Cortex-A72 has 2 integer and 2 fp units. Other modern cores have 3 integer units and 2 fp units. For both, those settings should be more realistic. I can't run any benchmarks at the moment though, as I am travelling.


https://reviews.llvm.org/D43079