<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="margin:0px">It is possible to make this TD file follow current microchips device list definition ? I am glad to take this work.</div></div></div></div></div></blockquote><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="margin:0px">2. add new production devices</div></div></blockquote><div><br></div><div>This sounds fine to me - missing additions to the device list are welcome. <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="margin:0px">1. remove EOL devices</div></div></blockquote><div><br></div><div>I don't think there is value in removing definitions for EOL devices, as the cost of maintenance is effectively zero.</div><div><br></div><div>If there are EOL features (that is, a feature that is only implemented on old EOL devices and no current), then there may be value in removing said feature and the logic that drives it, but it should be done on a case-by-case basis. The maintenance burden lies within the implementation of the actual old-chip features, not the TableGen definitions that enable particular old-chip features for particular old-chips, for example.</div><div><br></div><div>Regards,</div><div>Dylan<br></div><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 16, 2020 at 10:24 AM Fangrui Song <<a href="mailto:maskray@google.com">maskray@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2020-09-15, Ben Shi via llvm-dev wrote:<br>
>Hi,<br>
><br>
><br>
>The AVRDevice.td of the AVR target seems too old, some devices like at90s have already been End-Of-Live, and some devices such as AVR DA family are not included.<br>
><br>
><br>
>It is possible to make this TD file follow current microchips device list definition ? I am glad to take this work.<br>
><br>
><br>
>1. remove EOL devices<br>
>2. add new production devices<br>
><br>
><br>
>Ben<br>
<br>
CC the code owner of the AVR backend.<br>
</blockquote></div>