[PATCH] D52785: [PseudoSourceValue] New category to represent floating-point status

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle added a comment.

FWIW, I also think it's important not to drop MachineMemOperands. As Hal points out, MMOs carry non-optional information such as volatile-ness. And the Size is non-optional information at least for the AMDGPU backend.

If the machine outliner does not preserve MMOs, maybe it'd be a good idea to try to understand why that is? And explicitly allow dropping selected parts of them like Hal suggests?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D52785