[PATCH] D70874: [X86] Add initialization of MXCSR in llvm-exegesis

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 3 23:39:51 PST 2019


gchatelet added a comment.

In D70874#1764713 <https://reviews.llvm.org/D70874#1764713>, @courbet wrote:

> Regarding whether masking instruction is the right thing to do: Yes for now, because this will ensure that these instructions will at least pass, but on the other hand that might not be perfectly representative of the real performance characteristics. AFAICT @gchatelet still wants to work on value constraints & exploration.


Yes definitely. LGTM as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70874





More information about the llvm-commits mailing list