ext/pb_ds/regression/trie_no_data_map_rand

[Paolo Carlini]

Hi,

I suspect you meant to CC another Paolo... ;)

Anyway:

> I was trying to track down what PRE is doing to these testcases
> that causes them to crash, and I noticed that while I can't get
> them to crash without turning on PRE (IE -fno-tree-pre "fixes" them),
> they seem to have a *massive* number of valgrind failures consisting
> of "invalid reads" even with PRE turned off.

I don't think those failures are normal, but Benjamin knows that code *much* better than me. While we are at it, I think those testcases take far too much to compile and should be split to simpler units (probably also easier to debug, then ... ;)

If, while waiting for Benjamin feedback, you could debug a bit more, it would be great, because, all-in-all, the chances that only the new PRE triggers undefined behavior seem pretty small to me, just a guess of course.

Paolo.