[PATCH] D43630: [Driver] Fix search paths on x32

John Paul Adrian Glaubitz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 13 11:59:31 PDT 2018


glaubitz added a comment.

I don't think that "test/Driver/linux-header-search.cpp" needs to be updated. x32 is actually supported in both an x86_64-linux-gnu environment and x86_64-linux-gnux32 environment. In order to update test/Driver/linux-header-search.cpp, we would have to add a debian-10-x32 chroot to the LLVM test environment. However, the patch by @jrtc27 actually doesn't break the previous tests as his patch just allows the native header and library paths as altnernative search paths.

The same applies to test/Driver/cross-linux.c. The x32 test there will still remain valid, even with the changes.


Repository:
  rC Clang

https://reviews.llvm.org/D43630





More information about the cfe-commits mailing list