[PATCH] D60659: [InstCombine] Eliminate stores to constant memory

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 30 16:23:49 PDT 2019


reames added a comment.

In D60659#1483597 <https://reviews.llvm.org/D60659#1483597>, @liutianle wrote:

> hi reames, I have a question in the view of customer. If he writes deliberately to a read-only memory to expect trigger a seg fault, or if he wants to write to a memory which he does not realize it is read only. Should compiler tell him the writing is invalid, rather than treat it as a nop silently?


See the discussion in the commit thread about this being undefined behaviour and possibilities for sanitization.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60659





More information about the llvm-commits mailing list