[PATCH] D42725: IRGen: Move vtable load after argument evaluation.

2018-02-05 Thread Peter Collingbourne via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL324286: IRGen: Move vtable load after argument evaluation. (authored by pcc, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D42725?vs=132094&i

[PATCH] D42725: IRGen: Move vtable load after argument evaluation.

2018-01-30 Thread Peter Collingbourne via Phabricator via cfe-commits
pcc created this revision. pcc added reviewers: vlad.tsyrklevich, rsmith. This change reduces the live range of the loaded function pointer, resulting in a slight code size decrease (~10KB in clang), and also improves the security of CFI for virtual calls by making it less likely that the function