This LGTM, but please wait for the OK of a more experienced reviewer before committing.
Cheers, James On 1 May 2012 09:32, 陳韋任 <[email protected]> wrote: > Hi Eli, > > James sugguests me that we should use name ITANIUM rather than X86 since we > are dealing with Itanium C++ ABI (used by x86) and ARM ABI. Please use the > attach new patch instead of the old one. Thanks! > >> This patch addresses x86/ARM C++ ABI difference on member function pointer. >> You can see more detail in Sec. 3.2.1 "Representation of pointer to member >> function" [1]. >> >> Regards, >> chenwj >> >> [1] >> http://infocenter.arm.com/help/topic/com.arm.doc.ihi0041c/IHI0041C_cppabi.pdf > > Regards, > chenwj > > -- > Wei-Ren Chen (陳韋任) > Computer Systems Lab, Institute of Information Science, > Academia Sinica, Taiwan (R.O.C.) > Tel:886-2-2788-3799 #1667 > Homepage: http://people.cs.nctu.edu.tw/~chenwj _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
