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

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 1 02:17:49 PST 2019


pengfei added a comment.

> I'm not sure I understand why this needs to be initialized. Why don't we need to do it for FPCW?

I'm sorry for not giving comments. Yes, we do need for FPCW if this is a reasonable approach. But I'm not sure if we need to explicatly initialize registers like MXCSR and FPCW or not either.
Can we just return a `MCInstBuilder(X86::NOOP)` for them to make the check pass (In Assembler.cpp L44, it judges the `IsSnippetSetupComplete` by checking if the return value is empty).


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