[PATCH] D131290: [llvm-objdump] Create name for fake sections
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 6 00:54:20 PDT 2022
jhenderson added a comment.
In D131290#3771385 <https://reviews.llvm.org/D131290#3771385>, @namhyung wrote:
> SmallVector<std::string, 0> SecNames;
> ...
> SecNames.push_back(("PT_LOAD#" + Twine(Idx)).str());
>
> This makes the test failing again.
Please see the second half of my previous comment re. vector resizing (can apply to SmallVector too potentially). I would expect that to be a problem.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131290/new/
https://reviews.llvm.org/D131290
More information about the llvm-commits
mailing list