This is an alternative to D129154. See discussions on https://discourse.llvm.org/t/fast-scalar-fsqrt-tuning-in-x86/63605
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
This comment in X86.td is wrong now
// NOTE: As a default tuning, "generic" aims to produce code optimized for the // most common X86 processors. The tunings might be changed over time. It is // recommended to use "x86-64" in lit tests for consistency.
llvm/lib/Target/X86/X86TargetMachine.cpp | ||
---|---|---|
257–262 | Probably worth a comment here |
Comment Actions
This patch will make my proposed change in D129154 unnecessary, but can you copy the sqrt-fastmath-tunecpu-attr.ll (with a change to the @f32_tune_x86_64 test to reflect the current tuning when "tune-cpu"="x86-64" is used)?
Probably worth a comment here