[PATCH] D73582: [AMDGPU] override isHighLatencyDef

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 29 02:34:31 PST 2020


foad added a comment.

I have no objection to this patch, but I'm not sure isHighLatencyDef will ever be called by generic code. I think the latency information ususally comes from the sched model instead. Also note that TargetInstrInfo::defaultDefLatency never calls isHighLatencyDef on a load instruction!


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

https://reviews.llvm.org/D73582





More information about the llvm-commits mailing list