[PATCH] D69915: [AMDGPU] Fix bug introduced in 47a5c36b37f0

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov marked 2 inline comments as done.
dfukalov added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:741
     if (Shuffle->changesLength())
-      return -1;
+      BaseT::getUserCost(U, Operands);
 
----------------
arsenm wrote:
> I don't see how this works? Shouldn't this be returning instead of falling through?
You're right, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69915