fhahn added reviewers: fhahn, efriedma, hfinkel.
fhahn added a comment.

This seems similar to the problem described in 
https://bugs.llvm.org/show_bug.cgi?id=46210 : for intrinsics that are marked as 
reading memory, we do not account for the fact that the memory could be 
modified outside of the current program, which is what happens here.

The intrinsic seems like a good approach for addressing the issue for 
llvm.read_register incrementally. I guess there are other instances where we 
actually want ` llvm.read_volatile_register` on other platforms as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80911



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to