[PATCH] Relax assertion in x86_64 byval argument handling for 32-bit pointers

[Derek Schuff]

================
Comment at: lib/CodeGen/TargetInfo.cpp:2501
@@ -2500,3 +2500,3 @@
   if (HiStart != 8) {
     // There are only two sorts of types the ABI generation code can produce for
     // the low part of a pair that aren't 8 bytes in size: float or i8/i16/i32.
----------------
jvoung wrote:
> For the old "two sorts of types..." comment, technically a pointer type is still a separate type from i8/i16/i32, so it would be worth updating the comment for x32 pointers.
> 
> I assume that the rest of the code knows how to cast a pointer type to the wider i64 type.
Updated the comment.
And yes, the rest of the code handles this case just fine.

http://reviews.llvm.org/D10701

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/