[PATCH] Fix LLVMSetMetadata for MDNodes that contain a single value

Duncan Exon Smith dexonsmith at apple.com
Tue Jan 27 07:25:07 PST 2015


AFK, sorry for the terseness. 

Looks correct; thanks for working on this. 

- Testcase?
- All the ternary operators are hard to read.  I'd switch to ifs and early returns. 
- Unwrapping Val twice is a little confusing (and, unnecessary). 

-- dpnes

> On Jan 27, 2015, at 1:56 AM, Björn Steinbrink <bsteinbr at gmail.com> wrote:
> 
> I guess I should mention that triggering the assertion is a regression from 3.5.
> 
> 
> http://reviews.llvm.org/D7165
> 
> EMAIL PREFERENCES
>  http://reviews.llvm.org/settings/panel/emailpreferences/
> 
> 




More information about the llvm-commits mailing list