Skip to content

Rename strip reflect to strip nonsemantic#4661

Merged
s-perron merged 4 commits intoKhronosGroup:masterfrom
s-perron:reflect_rename
Dec 15, 2021
Merged

Rename strip reflect to strip nonsemantic#4661
s-perron merged 4 commits intoKhronosGroup:masterfrom
s-perron:reflect_rename

Conversation

@s-perron
Copy link
Copy Markdown
Collaborator

In #3110, the strip reflect
pass was changed to also remove all explicitly nonsemantic instructions. This
makes it so that the name of the pass no longer reflects what the pass actually
does. This change renames the pass so that it reflects what the pass actaully does.

In KhronosGroup#3110, the strip reflect
pass was changed to also remove all explicitly nonsemantic instructions.  This
makes it so that the name of the pass no longer reflects what the pass actually
does.  This change renames the pass so that it reflects what the pass actaully does.
@s-perron s-perron requested a review from dneto0 December 14, 2021 18:44
@s-perron s-perron requested a review from dneto0 December 14, 2021 18:53
Copy link
Copy Markdown
Collaborator

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

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

Thanks!

@dneto0
Copy link
Copy Markdown
Collaborator

dneto0 commented Dec 14, 2021

Have to update the Android.mk file to react to the renamed .cpp file.

@s-perron s-perron merged commit 354a46a into KhronosGroup:master Dec 15, 2021
@s-perron s-perron deleted the reflect_rename branch December 15, 2021 14:55
@s-perron s-perron mentioned this pull request Dec 15, 2021
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.

2 participants