<table><tr><td style="">phosek added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://reviews.llvm.org/D25067" rel="noreferrer">View Revision</a></tr></table><br /><div><p>LGTM! Shouldn't <tt style="background: #ebebeb; font-size: 13px;">TOOL_IS_TOOLCHAIN</tt> be lowercase since a lower-case variable (I've seen both variants used in LLVM so I'm not sure if there is convention)? Shall I just update the diff with your version or do you want submit a new diff?</p></div><br /><div><strong><strong>Repository:</strong></strong><div><div>rL LLVM</div></div></div><br /><div><a href="https://reviews.llvm.org/D25067" rel="noreferrer">https://reviews.llvm.org/D25067</a></div><br />