<div dir="ltr">It's definitely doable, but I'd be worried about the maintenance burden. Beyond contributing the initial patches, I'd want to see a maintenance commitment and relatively comprehensive tests that can be run upstream.<div><br></div><div>For example, if there were an i24 MVT, how would I test my target independent SDAG change that operates on all integer values? Currently, our answer to that question is "find a backend that uses it and test that". Without such a backend, it's hard for us to promise that this support will continue to work.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 3:12 AM, Tyro Software <span dir="ltr"><<a href="mailto:softwaretyro@gmail.com" target="_blank">softwaretyro@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-size:12.8000001907349px">Back in 2009 there was some discussion of the practicality of supporting char sizes greater than 8-bit:<br></div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px"><div><a href="http://lists.cs.uiuc.edu/pipermail/cfe-dev/2009-September/thread.html#6349" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-dev/2009-September/thread.html#6349</a></div><div><a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2009-September/thread.html#26025" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2009-September/thread.html#26025</a></div></div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">with the consensus seemingly being "quite doable, please get a good patch and submit".</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">However the current code appears (to my neophyte eyes) to be explicitly 8-bit, e.g. one instance called out in the mail thread remains:</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px"><div>/// isString - This method returns true if this is an array of i8.</div><div>bool ConstantDataSequential::isString() const {</div><div>  return isa<ArrayType>(getType()) && getElementType()->isIntegerTy(8);</div><div>}</div></div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">I didn't find anything related beyond this mail thread such as a discussion of a patch but of course I might be searching too narrowly - perhaps someone here can recall whether it went any further, whether insurmountable barriers do exist, etc?</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">Thanks for whatever advice & thread necromancy you can offer,</div><div style="font-size:12.8000001907349px">Tyro</div></div>
<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></blockquote></div><br></div>