[PATCH] D44608: [x86] put nops into the WriteNop class and customize for Jaguar

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 19 02:15:06 PDT 2018


courbet accepted this revision.
courbet added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D44608#1041238, @spatel wrote:

> 2. Move 'fnop' from the default WriteMicrocoded class to WriteNop. This may not be completely accurate for Jaguar, but it's still closer to the truth (and also for Silvermont based on Agner's tables).


I agree that this is likely more accurate latencywise, and that WriteNop is a very reasonable default. Intel scheduling models override it anyway for each microarchitecture.


https://reviews.llvm.org/D44608





More information about the llvm-commits mailing list