<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 6, 2014 at 9:25 AM, Rafael EspĂ­ndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><div class="h5">On 25 April 2014 11:03, Dave Johansen <<a href="mailto:davejohansen@gmail.com">davejohansen@gmail.com</a>> wrote:<br>

> I opened a Bugzilla requesting that support for Amazon Linux be added (<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=19550" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=19550</a> ). I've attached the patch for<br>
> 3.4 and 3.5 that add the appropriate triple values for Amazon Linux.<br>
<br>
</div></div>We normally make sure the change is in trunk and then backport, so<br>
commenting only on the 3.5 change:<br>
<br>
Can you add a testcase? Under clang/test/Driver/Inputs/ you will find<br>
some fake directory layouts, you should be able to add one for amazon<br>
linux and add a test that uses it.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div><div class="gmail_extra">For the sake of documentation, I started another thread with [PATCH] in the subject since this one hadn't gotten any responses. It can be found at:<br><a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20140428/104530.html">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20140428/104530.html</a><br>
<br></div><div class="gmail_extra">And here's a repost of what I posted in that thread:<br></div><div class="gmail_extra"><br><div>I'm just the maintainer of the llvm package for RHEL/EPEL 6 and I 
was just trying to be a good citizen and flow the patch upstream. I 
don't have access to Amazon Linux to test anything and don't use it so I
 don't have the time/interest to make a test.<br>
</div>Sorry,<br>Dave<br></div></div>