[PATCH] D70881: [X86] Model MXCSR for all AVX512 instructions

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 17:26:37 PST 2019


pengfei added a comment.

In D70881#1766211 <https://reviews.llvm.org/D70881#1766211>, @craig.topper wrote:

> In D70881#1766203 <https://reviews.llvm.org/D70881#1766203>, @pengfei wrote:
>
> > In D70881#1765349 <https://reviews.llvm.org/D70881#1765349>, @craig.topper wrote:
> >
> > > I think SAE instructions and embedded rounding instructions still read the DAZ and FTZ bits from MXCSR
> >
> >
> > I think so, but currently we don't model DAZ and FTZ, right?
>
>
> The comment in X86RegisterInfo.td says that, but where was that discussed?


You are correct, we do need to model DAZ and FTZ. I think I commented "not modeled" just for simplification. I will commit a patch to model them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70881





More information about the llvm-commits mailing list