[PATCH] D89178: [X86] Alternate implementation of D88194.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

Are we ok with this implementation?

I don't have a great way to protect intrinsics not using TargetConstant properly in the future, but I think we're ok at the moment based on what I see in the isel table.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89178