[PATCH] D142782: [AMDGPU] Add basic support for extended i8 perm matching

[Jeffrey Byrnes via Phabricator via llvm-commits]

jrbyrnes marked an inline comment as done.
jrbyrnes added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:9766
+/// value of the byte is either constant zero or comes from memory.
+struct ByteProvider {
+  // For constant zero providers Load is set to nullptr. For memory providers
----------------
arsenm wrote:
> arsenm wrote:
> > Can you keep this as a generic utility?
> By generic utility I mean in generic code and used by the load combine as well
Hey Matt -- thanks for comments. I think I don't fully understand this one -- I guess you didn't mean https://reviews.llvm.org/D143018 ? By generic, do you mean templated base class (perhaps in ADT) ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142782