search for: gettgtmemintrinsic

From: Villmow, Micah Sent: Tuesday, November 06, 2012 1:37 PM To: 'llvm-dev at cs.uiuc.edu' Cc: Guo, Xiaoyi Subject: Bug in SelectionDAG visitTargetIntrinsic We ran into a problem where specifying IntrNoMem was causing our instruction selection to fail with target specific intrinsics. After looking into the code and ISel debug it looks like tablegen and SelectionDAG are using different

...gned Intrinsic) { - bool HasChain = !I.doesNotAccessMemory(); - bool OnlyLoad = HasChain && I.onlyReadsMemory(); + unsigned Intrinsic) { + // Info is set by getTgtMemInstrinsic + TargetLowering::IntrinsicInfo Info; + bool IsTgtIntrinsic = TLI.getTgtMemIntrinsic(Info, I, Intrinsic); + bool HasChain = Info.readMem || Info.writeMem; + bool OnlyLoad = HasChain && Info.readMem; This doesn't seem right. If a call is marked ReadNone, it doesn't seem legal to select it to an intrinsic that read / write memory. By definition a "ReadNone&quo...

All, The attached patch adds an extra hook to TargetLowering that allows a target to change the Intrinsic ID and Values associated with an intrinsic call during SelectionDAG construction. The existing getTgtMemIntrinsic hook allows a target to change the Intrinsic ID but not the associated Values. Further, we wish to use this hook for more than just memory-touching intrinsics. We would like to get the community's feedback on this so as to make sure this patch is as universally applicable as possible. While...

...Cast 13 llvm::isSafeToLoadUnconditionally 13 isVectorPromotionViableForSlice 12 llvm::ARMTargetLowering::getTgtMemIntrinsic 12 getVectorType 12 llvm::ConstantFoldGetElementPtr 11 llvm::IRBuilderBase::CreateMaskedL...