[PATCH] D112016: [IR] Introduce load assume operand bundle

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 18 11:26:58 PDT 2021


aeubanks added a comment.

In D112016#3071078 <https://reviews.llvm.org/D112016#3071078>, @jdoerfert wrote:

>> This is useful for exposing the vtable value of a C++ object after its constructor without having to insert a load into the instruction stream.
>
> Why is an assume better than a load?

Adding an extra load will affect cost modelling, e.g. inliner thresholds, and other various ad-hoc cost heuristics. assumes are free.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112016/new/

https://reviews.llvm.org/D112016



More information about the llvm-commits mailing list