[PATCH] D87888: [X86] Use inlineasm flag output for the _bittest* intrinsics.

[Craig Topper via Phabricator via cfe-commits]

craig.topper added a comment.

In D87888#2285878 <https://reviews.llvm.org/D87888#2285878>, @rnk wrote:

> Honestly, I forget exactly what the memory clobber does beyond the "sideeffect" marker. I would expect LLVM to model these just as external function calls that could read or write memory that is passed to them.

The middle end may not care about the memory clobber. I think in the backend the memory clobber will cause mayLoad and mayStore to be set on the INLINASM instruction. Not sure if that does anything that sideeffects didn't already do.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87888