| commit | 2648a77f0499b2aeb3d624a1b44d2f1d2dd01835 | [log] [tgz] |
|---|---|---|
| author | Paul Pluzhnikov <ppluzhnikov@google.com> | Wed Nov 18 17:55:01 2009 -0800 |
| committer | Arun Sharma <aruns@google.com> | Fri Nov 20 12:04:13 2009 -0800 |
| tree | 7e81f5f9b461bf0ca734c36fbe0f3cf26dc813a9 | |
| parent | f958b9dc4f19c94281b2679b3a6bbb39828beb90 [diff] |
Simplify control flow Greetings, Attached patch gets rid of additional unnecessary branch (rs_get_cache can not return NULL unless caching_policy is UNW_CACHE_NONE), gets rid of goto's, and makes apply_reg_state (major CPU consumer) execute with cache lock not held (before the patch, apply_reg_state was called with lock held for newly-inserted entries, but not for found-in-cache entries). Tested on Linux/x86_64 with no regressions. Thanks, -- Paul Pluzhnikov