<div>LGTM.</div><div><br></div><div>Thanks,</div><div>  - Daniel</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 1, 2012 at 10:49 AM, Kristof Beyls <span dir="ltr"><<a href="mailto:kristof.beyls@arm.com" target="_blank">kristof.beyls@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Daniel,<br>
<br>
The attached one-line patch allows me to use LNT to start a run of the test<br>
suite with GCC as a compiler on an ARMv7-linux platform.<br>
<br>
My understanding is that LNT tries to figure out what the cc1 binary is by<br>
calling the compiler (e.g. /usr/bin/gcc) as follows:<br>
/usr/bin/gcc -v -E -x c /dev/null -###<br>
<br>
On an x86 machine, this seems to print out a number of lines, including the<br>
following, that is parsed by LNT:<br>
 "/usr/lib/gcc/x86_64-linux-gnu/4.4.3/cc1" "-E" "-quiet" "-v" "/dev/null"<br>
"-D_FORTIFY_SOURCE=2" "-mtune=generic" "-fstack-protector"<br>
<br>
On the ARMv7-linux machine I tried it on, the above command results in the<br>
following line containing cc1 to be printed:<br>
 /usr/lib/gcc/arm-linux-gnueabihf/4.6/cc1 -E -quiet -v -imultilib .<br>
-imultiarch arm-linux-gnueabihf /dev/null "-march=armv7-a"<br>
"-mfloat-abi=hard" "-mfpu=vfpv3-d16" -mthumb -fstack-protector<br>
<br>
The regular expression used in LNT to extract the cc1 path is the following,<br>
which does expect the double quotes to always be present around cc1:<br>
r' "([^"]*)".*"-E".*'<br>
<br>
The attached patch adapts the regular expression to correctly parse the<br>
cc1-line whether the double quotes are present or not.<br>
<br>
This is my first patch to LNT, so even though it seems trivially simple,<br>
could you review it before I commit?<br>
<br>
Thanks,<br>
<br>
Kristof<br>
</blockquote></div><br></div>