[PATCH] D94268: Allow _mm_empty() (via llvm.x86.mmx.emms) to be a no-op without MMX.

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 19:31:25 PST 2021


pengfei added a comment.

Is inline assembly the only case `emms` instruction will be needed? But inline assembly doesn't enable `mmx` attribute automatically, right? E.g. https://godbolt.org/z/43ases
Analyzing asm block and appending the `mmx` attribute if we see `mmx` instructions might be needed. But if we do the analysis, just adding an `emms` instruction at the end of the block seems better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94268



More information about the llvm-commits mailing list