[llvm-commits] Patch: avoid legalization assert

Mon Ping Wang monping at apple.com
Thu Jan 29 23:19:34 PST 2009


Hi,

In the test case below, we will generate an assert
   "Illegal type introduced after type legalization?"

In this test case, when legalizing the divide operation, we end up  
generating a library call with nodes that needs type legalization.   
This patch avoids the legalize assert if we run LegalizeType and then  
run LegalizeDAG.  This is a stop gap fix because we don't want to do  
any  type legalization to be done in LegalizeDAG. When we legalize an  
operation that generates a call to a library function, maybe we should  
call LegalizeType on the call sequence nodes that were created for the  
call.

-- Mon Ping

-------------- next part --------------
A non-text attachment was scrubbed...
Name: avoid.patch
Type: application/octet-stream
Size: 2381 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090129/acc7635d/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: legalize_assert.ll
Type: application/octet-stream
Size: 211 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090129/acc7635d/attachment-0001.obj>


More information about the llvm-commits mailing list