[PATCH] D53174: X86/TargetTransformInfo: Report div/rem constant immediate costs as TCC_Free

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

This totally makes sense, but why is ConstantHoisting pulling those constants out of the only block that uses them? I thought it tried to keep constants in the nearest dominator of the uses?


Repository:
  rL LLVM

https://reviews.llvm.org/D53174