[PATCH] D76792: [InlineFunction] Update metadata on loads that are return values

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 01:03:45 PDT 2020


nikic added a comment.

For the record, this causes a 0.6-0.8% compile-time regression <http://llvm-compile-time-tracker.com/compare.php?from=0d71782f4e29784873080571f7aebd5bdd163f3f&to=1d0f757904919d19f1cf5dcd307874bceb1e9efb&stat=instructions> on the tramp3d-v4 benchmark (there is also a 0.03% code-size improvement, so at least something is happening there codegen-wise). Can't say whether this is because adding the metadata itself has some cost, or because the presence of the metadata makes further optimization more expensive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76792





More information about the llvm-commits mailing list