[llvm-commits] [PATCH] Restructure a part of SimplifyCFG

Matthijs Kooijman matthijs at stdin.nl
Thu May 22 09:07:42 PDT 2008


Hi All,

> The patch looks great to me, if it passes tests, please apply.
I've pulled the patch through the (nightly) test bench a few times and there were no
extra failed tests. I also couldn't find any reproducible performance
degradations in the llc and jit runs. 

It does seem that some of the cbe tests suffer some degradation, but I'm not
sure how important that is. In the particular case I looked at, the code was
only restructured a little bit, but the removal of a loop preheader resulted
in a single do { } while(1) structure in the output where there used to be
two. I guess this throws off gcc's optimizer when recompiling the C code.

I'll do a last full test run tonight (to do one thorough check of llc
performance) and commit tomorrow morning.

Gr.

Matthijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20080522/4d757c11/attachment.sig>


More information about the llvm-commits mailing list