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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D43630: ... Sylvestre Ledru via Phabricator via cfe-commits
    • [PATCH] D43... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D43... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D43... John Paul Adrian Glaubitz via Phabricator via cfe-commits

Reply via email to