dawn added a comment.

Hi Paul,

Some background...  I basically tried the same thing in my original 
http://reviews.llvm.org/D11102 and http://reviews.llvm.org/D11173.  In light of 
Sean's objections in http://reviews.llvm.org/D11173, I created  
http://reviews.llvm.org/D11482 which used an option to control this behavior, 
but Greg and Jim objected to that, so I took their comments and revised 
http://reviews.llvm.org/D11102 to what I believed the consensus was.

You should read the comments in http://reviews.llvm.org/D11482 as they apply 
here as well.

My revision of http://reviews.llvm.org/D11102 is what I believe the consensus 
was, so perhaps you would like to abandon this patch in favor of that one?

Thanks,
-Dawn


http://reviews.llvm.org/D11790



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

Reply via email to