[PATCH] D159529: [BOLT][YAML] Only read first profile per function
Amir Ayupov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 18 18:36:07 PDT 2023
Amir added inline comments.
================
Comment at: bolt/lib/Profile/YAMLProfileReader.cpp:307
+ }
+ BF = nullptr;
+ }
----------------
maksfb wrote:
> Unneeded.
It is needed because the attaching happens in lines 341-354. Preliminary exec count assignment is a suitable place where it's easy/cheap to check if the profile was attached already.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159529/new/
https://reviews.llvm.org/D159529
More information about the llvm-commits
mailing list