[PATCH] D122709: [RISCV] Enable cross basic block aware writevxrm insertion

[Kito Cheng via Phabricator via llvm-commits]

kito-cheng added a comment.

I think that's we treating different view for `vxrm`, ABI was said same as `frm`, but now we plan to pull that out due to controversial for now, but I still not convinced for that, we pull out that out for prevent making short-sighted ABI decision, but you go this way means we made decision on close the possibility of `venv`, and that made dynamic rounding mode more unpredictable IMO.

This way require reset by user instead of restore by compiler, that sound like you push the responsibility back to user, and that what compiler can do, it's really better way to go?  I am really suspect for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122709