labath added a reviewer: zturner.
labath added a comment.

Adding Zachary as he was about to remove this code.

The timing of this patch is a bit unfortunate, as it would complicate the 
removal of the code in question. If I am not mistaken, llvm's version currently 
does not support copy-on-write mappings, which is what we are trying to do 
here. The c-o-w could certainly be added there, and we will probably need to do 
it in the future, but I am not sure whether this is the right time for it. I 
don't want to impede Zachary's cleanup effort due to a "feature" that noone has 
serious commitment of supporting.


https://reviews.llvm.org/D30251



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

Reply via email to