Re: [cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Aldy Hernandez
On 10/20/11 15:56, Andrew MacLeod wrote: On 10/20/2011 04:54 PM, Aldy Hernandez wrote: Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? yes. btw, did you audit the other new

Re: [cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Andrew MacLeod
On 10/20/2011 04:54 PM, Aldy Hernandez wrote: Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? yes. btw, did you audit the other new expand routines to see if they handl

[cxx-mem-model] expand_atomic_load: Handle an empty target

2011-10-20 Thread Aldy Hernandez
Found this while testing the branch on ia64. The call to expand_val_compare_and_swap() above returns NULL_RTX when it can't find a suitable instruction. OK for branch? * optabs.c (expand_atomic_load): Handle a NULL target. Index: optabs.c ==