[LLVMdev] insertAfter method Patch

Evan Cheng echeng at apple.com
Mon Jan 12 22:38:33 PST 2009


Looks fine. Have you committed this?

Evan
On Jan 8, 2009, at 7:15 AM, Thomas Jablin wrote:

> John Criswell wrote:
>> Small nitpick: in Instruction.cpp, the comment for insertAfter says  
>> insertBefore and not insertAfter.
>> :)
>>
>> -- John T.
>>
>> _______________________________________________
>> LLVM Developers mailing list
>> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>>
>
> Thanks for spotting the problem. Looking at the code again, I made  
> the same mistake in the comment in Instruction.h.  Here is the patch  
> with the corrected comments.
> Tom
> <AddInsertAfter.patch>_______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev




More information about the llvm-dev mailing list