<p dir="ltr"><br>
On 4 Mar 2013 18:35, "Jack Carter" <<a href="mailto:jack.carter@imgtec.com">jack.carter@imgtec.com</a>> wrote:<br>
><br>
> Author: jacksprat<br>
> Date: Mon Mar  4 15:36:11 2013<br>
> New Revision: 176453<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=176453&view=rev">http://llvm.org/viewvc/llvm-project?rev=176453&view=rev</a><br>
> Log:<br>
> Mips specific inline assembler constraint 'R'<br>
><br>
> 'R' An address that can be sued in a non-macro load or store.<br>
> This patch includes a positive test case.<br>
><br>
><br>
> Modified:<br>
>     cfe/trunk/lib/Basic/Targets.cpp<br>
>     cfe/trunk/test/CodeGen/mips-constraint-regs.c<br>
><br>
> Modified: cfe/trunk/lib/Basic/Targets.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=176453&r1=176452&r2=176453&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=176453&r1=176452&r2=176453&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/lib/Basic/Targets.cpp (original)<br>
> +++ cfe/trunk/lib/Basic/Targets.cpp Mon Mar  4 15:36:11 2013<br>
> @@ -4486,6 +4486,9 @@ public:<br>
>      case 'x': // hilo register pair<br>
>        Info.setAllowsRegister();<br>
>        return true;<br>
> +    case 'R': // An address tha can be used in a non-macro load or store</p>
<p dir="ltr">Typo</p>
<p dir="ltr">> +      Info.setAllowsMemory();<br>
> +      return true;<br>
>      }<br>
>    }<br>
><br>
><br>
> Modified: cfe/trunk/test/CodeGen/mips-constraint-regs.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/mips-constraint-regs.c?rev=176453&r1=176452&r2=176453&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/mips-constraint-regs.c?rev=176453&r1=176452&r2=176453&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/test/CodeGen/mips-constraint-regs.c (original)<br>
> +++ cfe/trunk/test/CodeGen/mips-constraint-regs.c Mon Mar  4 15:36:11 2013<br>
> @@ -2,9 +2,7 @@<br>
>  // RUN: | FileCheck %s<br>
><br>
>  // This checks that the frontend will accept inline asm constraints<br>
> -// c', 'l' and 'x'. Semantic checking will happen in the<br>
> -// llvm backend. Any bad constraint letters will cause the frontend to<br>
> -// error out.<br>
> +// c', 'l' and 'x'.<br>
><br>
>  int main()<br>
>  {</p>
<p dir="ltr">Aren't you missing an actual test change here?</p>
<p dir="ltr">><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</p>