~smspillaz/folly/folly-git-master

Viewing all changes in revision 5688.

  • Committer: Facebook Github Bot
  • Author(s): Nathan Bronson
  • Date: 2018-07-17 15:37:41 UTC
  • Revision ID: git-v1:0aa15e94b22b7af103bd8be12e43ce161ddd0762
mark uses of folly::assume used to optimize placement new

Summary:
F14 currently has numerous places where folly::assume is used to
help optimization on GCC < 6.  These are all things the optimizer should
be able to deduce itself, so it would be nice to eventually remove them.
This diff marks them with TODO(T31574848), as well as removing two
occurrences in F14Table that duplicated an assume in F14Policy.

Reviewed By: yfeldblum

Differential Revision: D8849832

fbshipit-source-id: e88143b4d29f1a633b51467206c6fb6afd1cd296

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: