[LLVMbugs] [Bug 5251] New: getIntrinsicID() optimization

[bugzilla-daemon at cs.uiuc.edu]

http://llvm.org/bugs/show_bug.cgi?id=5251

           Summary: getIntrinsicID() optimization
           Product: new-bugs
           Version: trunk
          Platform: PC
        OS/Version: Windows NT
            Status: NEW
          Severity: normal
          Priority: P2
         Component: new bugs
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: nicolas at capens.net
                CC: llvmbugs at cs.uiuc.edu


Created an attachment (id=3675)
 --> (http://llvm.org/bugs/attachment.cgi?id=3675)
Initial implementation

The Function::getIntrinsicID() method is called all over the place, and in some
use cases can take a significant amount of total compilation time. Each time
it's called, it determines the intrinsic ID based on the function name. There's
great opportunity for optimization by storing the ID for later use.

The attached patch is an implementation of the optimization which adds an
intrinsicID field to the Function class and initialized it at construction
time. However, extra fields are unwanted, and this implementation assumes the
names of intrinsics do not change (though that assumption is already made in
the constructor when initializing that argument attributes of intrinsics).

Possibly the SubclassData field could be used for storing the intrinsic ID...


-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.