Skip to content

Do not access LoopKernel.schedule#393

Merged
inducer merged 2 commits into
mainfrom
avoid_knl_dot_schedule
Jun 3, 2021
Merged

Do not access LoopKernel.schedule#393
inducer merged 2 commits into
mainfrom
avoid_knl_dot_schedule

Conversation

@kaushikcfd

@kaushikcfd kaushikcfd commented Jun 3, 2021

Copy link
Copy Markdown
Collaborator
  • accessing LoopKernel.schedule warns about its deprecation
  • sed commands:
    git grep -l "l.schedule\b" | xargs sed -i
    "s/l.scheduled/linearization/g"
    cd test/
    git grep -l "get_one_scheduled_kernel" | xargs sed -i
    "s/get_one_scheduled/get_one_linearized/g"

Also updated LoopKernel.hash_fields

- accessing LoopKernel.schedule warns about its deprecation
- sed commands:
  git grep -l "l\.schedule\b" | xargs sed -i \
      "s/l.scheduled/linearization/g"
  cd test/
  git grep -l "get_one_scheduled_kernel" | xargs sed -i \
      "s/get_one_scheduled/get_one_linearized/g"
@kaushikcfd

Copy link
Copy Markdown
Collaborator Author

Warnings in the test suite go down from 82k to 932.

@inducer

inducer commented Jun 3, 2021

Copy link
Copy Markdown
Owner

82k

Lolsob. :)

@inducer inducer merged commit 6fc6dd8 into main Jun 3, 2021
@inducer inducer deleted the avoid_knl_dot_schedule branch June 3, 2021 17:28
@inducer

inducer commented Jun 3, 2021

Copy link
Copy Markdown
Owner

Thanks!

@inducer

inducer commented Jun 3, 2021

Copy link
Copy Markdown
Owner

For anyone reading this later, I reverted this via force-push because it caused lots of conflicts with #222.

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