[PATCH] D101474: [AMDGPU] Make some VOP3 insts commutable

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 02:31:52 PDT 2021


foad added a comment.

> This patch revises d35d8da7d6ac6c08578ec0569b072292631691e0 <https://reviews.llvm.org/rGd35d8da7d6ac6c08578ec0569b072292631691e0>.
> It contains the commute opportunities excluding float insts

Maybe mention D99376 <https://reviews.llvm.org/D99376> here rather than just the commit hash?

Why are you excluding float insts now? Is it related to the test failures that caused you to revert D99376 <https://reviews.llvm.org/D99376>, or is it a different problem?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101474



More information about the llvm-commits mailing list