[PATCH] D35084: [ConstHoisting] choose to hoist when frequency is the same

[Wei Mi via Phabricator via llvm-commits]

wmi created this revision.
Herald added a subscriber: sanjoy.

The patch is to adjust the strategy of frequency based consthoisting: Previously when the candidate block has the same frequency with the existing blocks containing a const, it will not hoist the const to the candidate block. For that case, now we change the strategy to hoist the const if only existing blocks have more than one block member. This is helpful for reducing code size.


Repository:
  rL LLVM

https://reviews.llvm.org/D35084

Files:
  include/llvm/Support/BlockFrequency.h
  lib/Transforms/Scalar/ConstantHoisting.cpp
  test/CodeGen/X86/constant-hoisting-bfi.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35084.105523.patch
Type: text/x-patch
Size: 4316 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170706/1fd2c82a/attachment.bin>