[Bug libstdc++/30204] std::vector operator[] 10x speedup (patch)

2006-12-14 Thread pcarlini at suse dot de
--- Comment #3 from pcarlini at suse dot de 2006-12-14 10:33 --- Likewise... -- pcarlini at suse dot de changed: What|Removed |Added Status|WAITING

[Bug libstdc++/30204] std::vector operator[] 10x speedup (patch)

2006-12-14 Thread pcarlini at suse dot de
--- Comment #4 from pcarlini at suse dot de 2006-12-14 11:18 --- By the way, forgot that in mainline and 4.2 branch the issue simply doesn't exist anymore: for completely different reasons, we are *already* using directly _M_start, _M_finish, and co. --

[Bug libstdc++/30204] std::vector operator[] 10x speedup (patch)

2006-12-14 Thread charles at rebelbase dot com
--- Comment #5 from charles at rebelbase dot com 2006-12-14 17:58 --- (In reply to comment #1) -O1 is enough to remove all advantages of this patch. Also, that isn't really a fair timing comparison, as you've removed the function call altogether (I still expect it to be faster, but

[Bug libstdc++/30204] std::vector operator[] 10x speedup (patch)

2006-12-13 Thread chris at bubblescope dot net
--- Comment #1 from chris at bubblescope dot net 2006-12-13 18:08 --- -O1 is enough to remove all advantages of this patch. Also, that isn't really a fair timing comparison, as you've removed the function call altogether (I still expect it to be faster, but possibly not by 10x) --

[Bug libstdc++/30204] std::vector operator[] 10x speedup (patch)

2006-12-13 Thread pinskia at gcc dot gnu dot org
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-12-14 02:49 --- Can you test with -O1 or above? As mentioned in comment #1, this should remove all advatages gained from the inlined version. Also, that isn't really a fair timing comparison, as you've removed the function call