<div dir="auto">My code is attached here. I m trying to add in x86 for my target in this initial phase. Maybe later i will use separate target.</div><div class="gmail_extra"><br><div class="gmail_quote">On Jul 19, 2017 10:04 PM, "Krzysztof Parzyszek" <<a href="mailto:kparzysz@codeaurora.org">kparzysz@codeaurora.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Could you share your .td file?<br>
<br>
Also, could you generate the .inc file by hand and show the output?<br>
<br>
You could use something like this to generate it:<br>
llvm-tblgen -gen-register-info -I.../lib/Target/<your-target> -I.../lib/Target -I.../include/llvm/Target <your-target>.td -o -<br>
<br>
What is the relationship of your target with X86?  Are you trying to add something to it, or are you working on a separate target?<br>
<br>
-Krzysztof<br>
<br>
On 7/19/2017 4:47 PM, hameeza ahmed wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I have made changes in 3 files:<br>
LaneBitmask.h, codegenregisters.cpp and miparser.cpp. files are attached here.<br>
<br>
Now i am getting following errors. which means registerinfo.inc file is not generated successfully.<br>
<br>
/PIM/lib/Target/X86/MCTargetDe<wbr>sc/X86BaseInfo.h:733:24: error:<br>
       no member named 'XMM8' in namespace 'llvm::X86'<br>
     if ((RegNo >= X86::XMM8 && RegNo <= X86::XMM31) ||<br>
<br>
<br>
fatal error: too many errors emitted, stopping now [-ferror-limit=]<br>
20 errors generated.<br>
<br>
When i comment out the line to construct 65536 bit register in <a href="http://registerinfo.td" rel="noreferrer" target="_blank">registerinfo.td</a> <<a href="http://registerinfo.td" rel="noreferrer" target="_blank">http://registerinfo.td</a>>. it run fine.<br>
<br>
What to do?<br>
<br>
On Thu, Jul 20, 2017 at 2:36 AM, Krzysztof Parzyszek <<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.org</a> <mailto:<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.or<wbr>g</a>>> wrote:<br>
<br>
    Those couldn't be done generically, that's why the asserts were added.<br>
<br>
    -Krzysztof<br>
<br>
    On 7/19/2017 4:30 PM, Craig Topper wrote:<br>
<br>
        What about the static asserts protecting a Log call and another<br>
        in the parser?<br>
<br>
        On Wed, Jul 19, 2017 at 2:26 PM Krzysztof Parzyszek<br>
        <<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.org</a> <mailto:<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.or<wbr>g</a>><br>
        <mailto:<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.or<wbr>g</a><br>
        <mailto:<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.or<wbr>g</a>>>> wrote:<br>
<br>
             On 7/19/2017 4:18 PM, Craig Topper wrote:<br>
              > LaneMask isn't as self contained as it should be. 64<br>
        bits is enough<br>
              > here. The problem is accidental leaking of the current size.<br>
              ><br>
              > For example there was a hard coded compare with 32 in<br>
        tablegen<br>
             until I<br>
              > fixed it recently.<br>
<br>
             This is most likely the only example.  I actually tested it<br>
        with<br>
             uint64_t (but without exceeding 32 lanes).<br>
<br>
             -Krzysztof<br>
<br>
             --<br>
             Qualcomm Innovation Center, Inc. is a member of Code Aurora<br>
        Forum,<br>
             hosted by The Linux Foundation<br>
<br>
        --         ~Craig<br>
<br>
<br>
    --     Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,<br>
    hosted by The Linux Foundation<br>
<br>
<br>
</blockquote>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<br>
</blockquote></div></div>