[PATCH] D89599: [AMDGPU] Fixed v_swap_b32 match

[Paul C. Anagnostopoulos via Phabricator via llvm-commits]

Paul-C-Anagnostopoulos reopened this revision.
Paul-C-Anagnostopoulos added a comment.
This revision is now accepted and ready to land.

When I build the latest version of the AMDGPU target, I get the warning shown below, which was treated as an error. I believe it started happening with this revision. I'm running on Windows.

Does it need to be:

  if (Size > 1 && (I->getNumImplicitOperands() > (I->isCopy() ? 0U : 1U)))

----------------------------------

...SIShrinkInstructions.cpp(551): warning C4018: '>': signed/unsigned mismatch


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89599