Evan:<br><br>Sorry for the late follow up, I was out of town last week.<br>Enclosed please find the updated patch including all<br>your suggestions and a dejagnus test.<br><br>Robert<br><br><div class="gmail_quote">On Thu, Jun 11, 2009 at 2:27 PM, Evan Cheng <span dir="ltr"><<a href="mailto:evan.cheng@apple.com">evan.cheng@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5"><br>
On Jun 8, 2009, at 2:42 PM, robert muth wrote:<br>
<br>
> On Sun, Jun 7, 2009 at 11:53 PM, Evan Cheng <<a href="mailto:evan.cheng@apple.com">evan.cheng@apple.com</a>><br>
> wrote:<br>
>><br>
>> On Jun 7, 2009, at 6:59 AM, robert muth wrote:<br>
>><br>
>>> On Sat, Jun 6, 2009 at 4:51 PM, Evan Cheng<<a href="mailto:evan.cheng@apple.com">evan.cheng@apple.com</a>><br>
>>> wrote:<br>
>>>> +cl::opt<std::string> FlagJumpTableSection("jumptable-section",<br>
>>>> +<br>
>>>> cl::init(".data.jtab"));<br>
>>>> +<br>
>>><br>
>>> I thought it would be nice to group all the jumptables together.<br>
>>> But as long as it stays configurable, I am fine to change the<br>
>>> default<br>
>>> to ".data".<br>
>><br>
>> There is already a TargetAsmInfo::JumpTableDataSection. Why not just<br>
>> use that?<br>
><br>
> Nice find. I will use that and possible change the current setting,<br>
> ".const", if it does not work,<br>
><br>
>>><br>
>>>> Is this necessary? Why not just put it in the normal data section?<br>
>>>> Also "outline" jumptable seems like a strange term. Can you think<br>
>>>> of<br>
>>>> something else?<br>
>>><br>
>>><br>
>>> Yes, that is a tough one. How about "NonInline" instead.<br>
>><br>
>> Or "OutOfLine"?<br>
><br>
> That works for me.<br>
><br>
>> Please add a test case as well. Thanks,<br>
><br>
> I am not sure how to go about testing.<br>
<br>
</div></div>For this please just add a test case to the llvm dejagnu tests.<br>
<br>
thanks,<br>
<font color="#888888"><br>
Evan<br>
</font><div><div></div><div class="h5"><br>
><br>
> I have  a script that compiles a bunch of test<br>
> programs (gnu c torture test, etc.) and then runs the executables<br>
> on qemu.  I run this script with and without my flags and<br>
> make sure that I do not introduce any new problems<br>
> -- there are currently plenty of vararg issues.<br>
><br>
> I was thinking of sending this script to the list<br>
> or maybe checking it into the llvm tree.<br>
><br>
> The key is that whatever tests there are they should just<br>
> be run with and without the new flag.<br>
> How do you run backend tests?<br>
><br>
> Robert<br>
><br>
><br>
><br>
><br>
>> Evan<br>
>><br>
>>><br>
>>> Robert<br>
>>><br>
>>>> Thanks,<br>
>>>> Evan<br>
>>>><br>
>>>> Sent from my iPhone<br>
>>>> On Jun 2, 2009, at 6:26 PM, robert muth <<a href="mailto:robert@muth.org">robert@muth.org</a>> wrote:<br>
>>>><br>
>>>> Hi:<br>
>>>><br>
>>>> This is my first patch submission. Hopefully, this is the proper<br>
>>>> the<br>
>>>> protocol.<br>
>>>> Attached is a patch for the llc ARM backend:<br>
>>>><br>
>>>> Added mechanism to generate switch table in a data section<br>
>>>> rather than having it interleaved with the code.<br>
>>>> This is controlled by command line flags and off by default.<br>
>>>> Also, tried to document and improve the code where I modified it.<br>
>>>><br>
>>>> Robert<br>
>>>><br>
>>>> <llc.patch.txt><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> LLVM Developers mailing list<br>
>>>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> LLVM Developers mailing list<br>
>>>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>>>><br>
>>>><br>
>>><br>
>>> _______________________________________________<br>
>>> LLVM Developers mailing list<br>
>>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>><br>
>> _______________________________________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>><br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br>