[llvm-commits] [llvm] r40128 - /llvm/trunk/test/CFrontend/2004-02-13-Memset.c

[Duncan Sands]

Hi Chris,

> This is fine.  However, I'm curious.  Do you mean it fails due to the  
> calls to get_pointer_alignment in TreeToLLVM::EmitBuiltinMemCopy?

Yes, the calls to get_pointer_alignment return zero.  This causes
EmitBuiltinMemCopy and friends to bail out.

> If so, we should fix TreeToLLVM::EmitBuiltinMemCopy to treat a null  
> return as alignment = 1.

A 0 return could also mean that the argument wasn't a pointer - I
will check if this is problematic or not.

Ciao,

Duncan.