[PATCH] D83913: [X86] Change the tuning settings for pentium4 to be more modern since its the default 32-bit cpu in clang

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 15:21:26 PDT 2020


craig.topper marked an inline comment as done.
craig.topper added a comment.

I'll probably wait a day to see if @rksimon or @spatel have any opinion



================
Comment at: llvm/lib/Target/X86/X86.td:1080
+
+  // Since 'pentirum4' is the default 32-bit CPU on Linux and Windows,
+  // give it more modern tunings.
----------------
MaskRay wrote:
> Typo?
thanks. I'll fix it before I commit.


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

https://reviews.llvm.org/D83913





More information about the llvm-commits mailing list