[mlir] Use affine dim instead of symbol in SCFToGPU lowering.
ClosedPublic
Actions

Authored by tpopp on Oct 20 2020, 2:35 AM.

Details

Reviewers

herhut
ftynse

Commits

rG72d5ac90b928: [mlir] Use affine dim instead of symbol in SCFToGPU lowering.

Summary

This still satisfies the constraints required by the affine dialect and
gives more flexibility in what iteration bounds can be used when
loewring to the GPU dialect.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

tpopp created this revision.Oct 20 2020, 2:35 AM

Herald added a project: Restricted Project. · View Herald TranscriptOct 20 2020, 2:35 AM

Herald added subscribers: rdzhabarov, tatianashp, msifontes and 14 others. · View Herald Transcript

tpopp requested review of this revision.Oct 20 2020, 2:35 AM

Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald TranscriptOct 20 2020, 2:35 AM

ftynse accepted this revision.Oct 20 2020, 2:38 AM

This revision is now accepted and ready to land.Oct 20 2020, 2:38 AM

Thanks!

This revision was landed with ongoing or failed builds.Oct 20 2020, 2:56 AM

Closed by commit rG72d5ac90b928: [mlir] Use affine dim instead of symbol in SCFToGPU lowering. (authored by Tres Popp <tpopp@google.com>). · Explain Why

This revision was automatically updated to reflect the committed changes.

Tres Popp <tpopp@google.com> added a commit: rG72d5ac90b928: [mlir] Use affine dim instead of symbol in SCFToGPU lowering..

Harbormaster completed remote builds in B75675: Diff 299311.Oct 20 2020, 3:16 AM

Revision Contents
Changeset List

Path

Size

mlir/

lib/

Conversion/

SCFToGPU/

SCFToGPU.cpp

9 lines

test/

Conversion/

SCFToGPU/

parallel_loop.mlir

36 lines

Diff 299311

View Options

mlir/lib/Conversion/SCFToGPU/SCFToGPU.cpp