[PATCH] D95580: [mlir][spirv] Add support for OpImageType

[Kareem Ergawy via Phabricator via llvm-commits]

ergawy added inline comments.


================
Comment at: mlir/lib/Target/SPIRV/Deserialization/Deserializer.cpp:1012
+  // TODO: Add support for Access Qualifier.
+  assert(!operands.empty() && "No operands for processing image type");
+  if (operands.size() != 8)
----------------
Weiwei-2021 wrote:
> ergawy wrote:
> > small nit: I think the following `if` condition is more appropriate for this check. We can remove this line.
> > 
> > (Hope you don't mind me passing by :) ).
> :) Thank you for reviewing!  I will remove this assertion and upload again (after I figure out how to upload a revision. This is my first time using this system - still learning. :p)
One way to do that is:

- Do whatever changes on your working copy (i.e. remove the `assert` in this case).
- `git add <file>` and `git commit --amend`.
- `arc diff origin/main`, arcanist should detect that you want to update this revision (rather than open a new one).

So, basically, updating a revision is the same flow as opening a new one with the exception of amending to the already existing commit (most of the time at least). Hope that helps.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95580