[PATCH] D99376: [AMDGPU] Mark additional VOP3 as commutable

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 14:31:01 PDT 2021


arsenm added a comment.

In D99376#2651640 <https://reviews.llvm.org/D99376#2651640>, @Joe_Nash wrote:

> It is unclear to me how to force a commute for testing purposes, but if anyone has an idea I can put that in. Vulkan short testlist passes fine with this patch. I will also test a compute psdb.

You could add a MIR test with two identical operations, other than commuting the operations and machine-cse should combine them


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99376



More information about the llvm-commits mailing list