This is an archive of the discontinued LLVM Phabricator instance.

[MLIR][SPIRVToLLVM] Updated documentation on entry points and not supported ops
ClosedPublic

Authored by georgemitenkov on Nov 20 2020, 10:48 AM.

Details

Summary

This patch addresses two issues:

  1. Not supported ops are updated to pick up the changes in the

SPIR-V dialect.

  1. Conversion on spv.ExecutionMode is updated.

Diff Detail

Event Timeline

georgemitenkov requested review of this revision.Nov 20 2020, 10:48 AM
ftynse added inline comments.Nov 23 2020, 1:38 AM
mlir/docs/SPIRVToLLVMDialectConversion.md
387

What's the LLVM equivalent? LLVM does not allow arrays without size.

georgemitenkov added inline comments.Nov 24 2020, 8:01 AM
mlir/docs/SPIRVToLLVMDialectConversion.md
387

I guess I didn't put it clear enough here! What I meant here is that we translate to

struct {
  int32_t executionMode;
};

if there are no values and to

struct {
  int32_t executionMode;
  int32_t values[];
};

if they exist.

georgemitenkov marked an inline comment as done.

Added a clearer explanation of how execution mode is
converted when there are no values associated with it.

antiagainst accepted this revision.Dec 1 2020, 11:50 AM
This revision is now accepted and ready to land.Dec 1 2020, 11:50 AM