[PATCH] D105836: [AMDGPU] Add TTI::isAmbientPhysRegUse() to allow VOP rematerilization

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 12 12:36:28 PDT 2021


arsenm added a comment.

Commit message should say TII



================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:131
+  // And implicit use of exec is not a real register read.
+  return MO.getReg() == AMDGPU::EXEC && MO.isImplicit();
+}
----------------
This also needs to verify it's a VALU use, you could still have an implicit exec read of a scalar instruction


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

https://reviews.llvm.org/D105836



More information about the llvm-commits mailing list