[PATCH] D81728: [InstCombine] Add target-specific inst combining

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D81728#2112158 <https://reviews.llvm.org/D81728#2112158>, @arsenm wrote:

> In D81728#2111901 <https://reviews.llvm.org/D81728#2111901>, @craig.topper wrote:
>
> > As far as I know and I might be wrong, but TargetTransformInfo up til now has only provided information. It doesn't do any transforms itself. Is adding transforms to it the right thing to do?
>
>
> This isn't strictly true. I recently added rewriteIntrinsicWithAddressSpace for example


I stand corrected then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81728