[PATCH] D68121: [X86] Model MXCSR for all SSE instructions
    Simon Pilgrim via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Sep 30 06:32:55 PDT 2019
    
    
  
RKSimon added a comment.
In D68121#1685783 <https://reviews.llvm.org/D68121#1685783>, @pengfei wrote:
> And we don't need to model MXCSR for (V)LDMXCSR/(V)STMXCSR, because they are using hasSideEffects and cannot be replaced due to we only modeling rounding and exception bits in MXCSR.
I'm not talking about removing the hasSideEffects tag (at least not yet), I'm just suggesting that it'd make sense to indicate that MXCSR is written/read by (V)LDMXCSR/(V)STMXCSR
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68121/new/
https://reviews.llvm.org/D68121
    
    
More information about the llvm-commits
mailing list