[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