Changed variable names to camel back style

This patch fixes the repeating index issue.

Refactoring

Make gpu.alloc op implement AllocLikeOpInterface

LGTM
Thanks for implementing the suggested changes.

Minor NFC changes.

Minor changes.

Addressed comments by nicolas.

Removed wrong changes in clang-format.sh

Addressed comments.

@nicolasvasilache I've made the required changes in the API. Also added tests featuring memref.load/store. However I've not handled folding related to vector transfer ops, so no tests involving those ops.

Addressed comments

I think this change alters the semantics, and won't work if the op has deallocate + some other effects on val. Those type of ops still qualify as dealloc ops.

I'm getting the following error:

@nicolasvasilache can you please take a look at my comments?

ping @nicolasvasilache can you please re-review?

Addressd review comments.

Addressed review comments.

Discarding unnecessary changes.

Rebased on latest master.

Addressed comments by @bondhugula.

Addressed comments by @bondhugula.

Addressed comments by @bondhugula.

Addressed comments by @bondhugula.

Fixed linting errors.

Added hasSingleEffect() as a part of SideEffectInterface.

Can you please provide the test case for which incorrect IR is generated?
I will provide the fix. Thanks in advance.

Removed trailing white spaces.

Fixed build issues.

Addressed review comments and rebased on latest master.

Adressed comments by @csigg, @bondhugula

ping @csigg, can you please take look at my comment?

Resolved build issues.

Addressed comments by @csigg.

Addressed comments by @bondhugula

Addressed comments by @bondhugula

Addressed comments by @bondhugula

Addressed review comments by @csigg.

Set hasVerifier field in gpu.memcpy removed earlier by mistake.

Fixed build errors.

ping @ JianyiCheng, can you please let me know if this revision has been abandoned? I actually require a similar kind of pass.

Initially Linalg::ConvOp had padding attribute, but the current set of convolution ops don't have them. Is there anyway I can define convolution with padding with the current Linalg Dialect?
CC - @bondhugula , @gysit , @nicolasvasilache

Added doc comments for the getMaxDimAndSymbol().

Rebased on latest master.

Rebased on latest master.

Removed extra parenthesis as suggested by @bondhugula.

@nicolasvasilache, @bondhugula and @rriddle, I have addressed all your comments. Please take a look.

Addressed comments by @nicolasvasilache and @bondhugula

ping @nicolasvasilache, I have addressed your comments. You can take a look.

Addressed comments by @nicolasvasilache and @bondhugula.

ping @rriddle and @nicolasvasilache can you please take a look at the replies of your respective comments.

Removed unnecessary #include<algorithm>.

Addressed comments.

Addressed comments by @bondhugula.

Fixed documentation @bondhugula .

ping @dcaballe and @springerm , can you please take a look and review this? This is a continuation of semi affine expression simplification PR.

Addressed all comments by @bondhugula

Added test cases.

ping @springerm and @dcaballe , can you please let me know whether the revision is ok or if any more changes are required to be made? Thanks in advance!

Added CHECK-LABEL in unit tests.

In D112808#3122209, @bondhugula wrote:

In D112808#3118161, @arnab-oss wrote:

Fixed a bug in the loop iterating over localExprs vector. Addressed comments
by @springerm.

Could you add a test case that exercises that part of the code? (if not already)

Added an example which is giving correct output now after the previous update bug fix.

Fixed a bug in the loop iterating over localExprs vector. Addressed comments
by @springerm.

Addressed all the comments by @bondhugula and @dcaballe.

Removed unwanted changes from clang-format.sh

Added an example showing the contents of indices, indexToExprMap and coefficients.

Addressed all the comments by springerm.