Denys Vlasenko wrote:
On Wednesday 07 July 2010 12:58, Maksym Kryzhanovskyy wrote:
Hi Denis,

can we use the __builtin_expect function in busybox?
-       if (!G.detect_link_func) {
+       if (__builtin_expect(G.detect_link_func == NULL, 0)) {

I don't see any changes in size, only obfuscated code.
I think the main point of __builtin_expect is to make the code faster by telling the compiler which branch is more likely, not to make the assembly code smaller or easier to read. Actually I would expect the above sequence to require an extra jump, making the code bigger.

Ralf
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to