[llvm-commits] [llvm] r57226 - in /llvm/trunk: include/llvm/Intrinsics.td include/llvm/IntrinsicsAlpha.td lib/Target/Alpha/AlphaISelDAGToDAG.cpp lib/Target/Alpha/AlphaISelLowering.cpp

[Chris Lattner]

On Oct 6, 2008, at 7:10 PM, Andrew Lenharth wrote:

> Author: alenhar2
> Date: Mon Oct  6 21:10:26 2008
> New Revision: 57226
>
> URL: http://llvm.org/viewvc/llvm-project?rev=57226&view=rev
> Log:
> Note that ADDC and company don't actually expand yet (missing in  
> legalize

Hi Andrew,

__builtin_alpha_umulh can be represented with generic LLVM IR (zext to  
i128, multiply, shr by 64, truncate to i64).  You should get good code  
for that sequence already, if not, that's a bug.

Can you just have llvm-gcc expand the builtin like we do for other  
builtins supported by LLVM IR?

-Chris