[PATCH] D50988: Remove Darwin support from POWER backend.

Kit Barton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 12:51:00 PDT 2018


kbarton added a comment.

In https://reviews.llvm.org/D50988#1206363, @MatzeB wrote:

> Actually can you explain the removal of `llvm/test/tools/dsymutil/PowerPC/sibling.test`?


I had an email conversation with @JDevlieghere regarding this test. Someone suggested a test based on an old version of llvm-gcc from an old Powerbook G4, which is how this test was created. That was the only reason for this being a PowerPC test.

I tried generating an alternate test using a new GCC on a ppc64le machine, but the test caused an Assert failure in dsymutil. Jonas is looking at. In the meantime, he said this specific test should not block this patch.

If you have another idea for generating an equivalent test, I'm happy to do it.


Repository:
  rL LLVM

https://reviews.llvm.org/D50988





More information about the llvm-commits mailing list