[PATCH] D84779: [AMDGPU] Add amdgpu specific loop threshold metadata

[Tim Corringham via Phabricator via llvm-commits]

timcorringham added a comment.

As I said in my reply to Matt, I made it target specific to avoid cluttering the llvm.loop metadata with something that is only used by one target. I am happy to make it llvm.loop.unroll.threshold if that is considered better.

However, I'm not really sure how generic it is as no other target has felt the need to do this (at least publicly). The function attribute is also amdgpu specific. Neither are used in any generic code, only in the target specific getUnrollingPreferences().

Is the consensus to change it to llvm.llop.unroll.threshold?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84779/new/

https://reviews.llvm.org/D84779