[llvm-commits] [llvm] r77747 - /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp
gohman at apple.com
Sat Aug 1 15:00:12 CDT 2009
On Jul 31, 2009, at 4:00 PM, Eli Friedman wrote:
> On Fri, Jul 31, 2009 at 3:41 PM, Bob Wilson<bob.wilson at apple.com>
>> Author: bwilson
>> Date: Fri Jul 31 17:41:21 2009
>> New Revision: 77747
>> URL: http://llvm.org/viewvc/llvm-project?rev=77747&view=rev
>> Allow target intrinsics that return multiple values, i.e., struct
>> in SelectionDAGLowering::visitTargetIntrinsic.
>> This removes a bit of special-case code for vector types. After
>> at it for a while, I managed to convince myself that it is not
>> The only case where TLI.getValueType() differs from MVT::getMVT is
>> for iPTR,
>> so this code could potentially make a difference for a vector of
>> But, it looks like that is not supported.
> Yes, a vector of pointers isn't legal; we only support integer and
> FP vectors.
And if that ever changes (:-)) TLI.getValueType(), which is what
ComputeValueVTs uses, would best handle pointer vectors itself,
so this commit is the right thing to do, regardless.
More information about the llvm-commits