[PATCH] D39053: [Bitfield] Add more cases to making the bitfield a separate location

[Strahinja Petrovic via Phabricator via cfe-commits]

spetrovic added a comment.

I tried to compile some important libraries for X86 and MIPS64 within Chromium with clang/llvm. I have compared results between LLVM trunk, and LLVM trunk with my patch. There is code size improvement on many libraries, here are some results:

- X86

libframe            ~1.5%
librendering        ~1.2%
liblayout_2         ~0.7%
libpaint_1          ~0.65%
libglslang          ~0.5%
libediting_3        ~0.5%
libcore_generated   ~0.5%
libanimation_1      ~0.5%

- Mips64

libglslang      ~3.2%
libframe        ~1.5%
liblayout_0     ~1.2%
libGLESv2       ~1%
librendering    ~0.7%
liblayout_1     ~0.5%
libcss_9        ~0.5%

Those are just some of libraries where we have code size improvement with this patch, I have also tried to compile LLVM with LLVM (with my patch) for x86 and MIPS64, and there is also code size improvement on both architectures. For example within clang executable for MIPS64 ~350 unaligned loads is removed. So, what do you think about it ?


https://reviews.llvm.org/D39053