Skip to content

Conversation

@kartcq
Copy link
Contributor

@kartcq kartcq commented Apr 16, 2025

Some of the changes in the patch include:

1. Using iterators instead of instruction pointers when applicable.
2. Modifying Polly functions to accept iterators instead of inst pointers.
3. Updating API usages such as use begin instead of front.

…ore areas

Some of the changes in the patch include:

	1. Using iterators instead of instruction pointers when applicable.
	2. Modifying Polly functions to accept iterators instead of inst pointers.
	3. Updating API usages such as use begin instead of front, and
	   moveBeforePreserving instead of moveBefore.

Change-Id: Iba4079efa64f64d81a0c87b826cc68012c8c87d6
Copy link
Contributor

@OCHyams OCHyams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one nit

Copy link
Member

@Meinersbur Meinersbur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kartcq kartcq merged commit 5f704f9 into llvm:main Apr 28, 2025
11 checks passed
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
…ore areas (llvm#135935)

Some of the changes in the patch include:

1. Using iterators instead of instruction pointers when applicable.
2. Modifying Polly functions to accept iterators instead of inst
pointers.
3. Updating API usages such as use begin instead of front.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants