[PATCH] D82437: [ELF] Add --dependency-file option
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 2 15:12:24 PDT 2020
phosek added a comment.
In D82437#2189738 <https://reviews.llvm.org/D82437#2189738>, @MaskRay wrote:
> CC @nemanjai who can test this patch on ppc64be.
I have also managed to reproduce the failure locally on x86_64 with assertions enabled. I don't understand the failure though. It seems like the insertion into the `SetVector` is failing because the element is already present (which shouldn't cause a failure), and if I insert a presence check before calling `insert`, it returns `false`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82437/new/
https://reviews.llvm.org/D82437
More information about the llvm-commits
mailing list