[cfe-dev] Target Definition for MicroBlaze

[Chris Lattner]

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.

-Chris