hokein added a comment.

> ! In http://reviews.llvm.org/D20360#432882, @aaron.ballman wrote:

>  World executable? That's a bit presumptuous. ;-) 0744 may be fine, but 0755 
> does not seem correct to me.

> 

> Regardless, I think that it should be a separate commit, not part of this one 
> as a drive-by (it has security implications, and that deserves review).


Has been removed now. Will sent a separate commit for it.


http://reviews.llvm.org/D20360



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

Reply via email to