[cfe-dev] Target Definition for MicroBlaze

Jakob Stoklund Olesen stoklund at 2pi.dk
Fri Feb 26 15:58:44 PST 2010


On Feb 26, 2010, at 1:12 PM, Chris Lattner wrote:

> 
> On Feb 26, 2010, at 11:39 AM, Wesley Peck wrote:
> 
>> On Feb 26, 2010, at 12:47 PM, Chris Lattner wrote:
>> 
>>> I'm not really thrilled with this.  Instead of creating an instance of MBlazeIntrinsicInfo, can't you get the TargetIntrinsicInfo* from the target machine in a target independent way?
>> 
>> I am a little bit confused about what you are asking here. Do you mean that I shouldn't use MBlazeIntrinsicInfo at all or that I should move its use to inside of the class MBlazeTargetInfo (inside of lib/Basic/Targets.cpp)?
> 
> I'm saying that it shouldn't *directly* use MBlazeIntrinsicInfo at all: Given a TargetMachine object (which will your MBlaze TargetMachine), clang should call the TM.getIntrinsicInfo() method, which will return the existing MBlazeIntrinsicInfo object without clang having to know about mblaze directly.

So probably getIntrinsicForGCCBuiltin() needs to be added to TargetIntrinsicInfo? It doesn't currently exist.

/jakob





More information about the cfe-dev mailing list