This is an archive of the discontinued LLVM Phabricator instance.

[Driver][test] Make RISCV tests robust with PATH=
ClosedPublic

Authored by sunshaoce on Jun 12 2022, 2:40 AM.

Details

Summary

When riscv64-unknown-linux-gnu-ld is in the PATH, clang -### -fuse-ld=ld --target=riscv64-unknown-linux-gnu will use unknown-linux-gnu-ld first, which causes the error in the lit test.

Diff Detail

Event Timeline

sunshaoce created this revision.Jun 12 2022, 2:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2022, 2:40 AM
sunshaoce requested review of this revision.Jun 12 2022, 2:40 AM

Why not use env PATH= in the lit run line to clear the PATH env variable?

Address @StephenFan's comment.

MaskRay accepted this revision.EditedJun 14 2022, 11:01 PM
MaskRay retitled this revision from [RISCV] Compatible with more RISCV related ld in the test to [Driver][test] Make RISCV tests robust with PATH=.

unknown-linux-gnu-ld

riscv64-unknown-linux-gnu-ld.

Please update your local commit message. I changed the patch's subject.

This revision is now accepted and ready to land.Jun 14 2022, 11:01 PM
This revision was landed with ongoing or failed builds.Jun 15 2022, 7:25 AM
This revision was automatically updated to reflect the committed changes.