This is an archive of the discontinued LLVM Phabricator instance.

[Driver] move NetBSD header search path management to the driver
ClosedPublic

Authored by brad on Jan 1 2023, 3:48 AM.

Diff Detail

Event Timeline

brad created this revision.Jan 1 2023, 3:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 1 2023, 3:48 AM
brad requested review of this revision.Jan 1 2023, 3:48 AM
brad edited the summary of this revision. (Show Details)
MaskRay accepted this revision.Jan 2 2023, 9:49 PM

Thanks!

This revision is now accepted and ready to land.Jan 2 2023, 9:49 PM
brad updated this revision to Diff 486505.Jan 5 2023, 2:31 AM

Tweak test.

This revision was landed with ongoing or failed builds.Jan 5 2023, 3:14 AM
This revision was automatically updated to reflect the committed changes.

Hi @brad ,

would you fix the tests
https://lab.llvm.org/buildbot/#/builders/60/builds/10125

looks like the problem is here:

// DRIVER-PASS-INCLUDES:      "-internal-externc-isystem" "/usr/include"

could be another path instead of "/usr/include".

this one should work for the most cases

// DRIVER-PASS-INCLUDES:      "-internal-externc-isystem" "{{.*}}/usr/include"

Hi @brad,
thank you for fixing the broken tests.

There is still a problem with netbsd.c test: https://lab.llvm.org/buildbot/#/builders/60/builds/10135/steps/9/logs/FAIL__Clang__netbsd_c
Here:

// DRIVER-PASS-INCLUDES-SAME: {{^}} "-internal-isystem" "[[RESOURCE]]{{/|\\\\}}include"
// DRIVER-PASS-INCLUDES-SAME: {{^}} "-internal-externc-isystem" "{{.*}}/usr/include"

The first's {{^}} breaks the test. It can be removed from the first line or from the both of them.
The tests are getting passed without this part on the win-x-cross builders. They should work on Linux/BSD hosts with these changes.
Would you check and update the tests accordingly?

The following changes are working on my cross builders

// DRIVER-PASS-INCLUDES-SAME: "-internal-isystem" "[[RESOURCE]]{{/|\\\\}}include"
// DRIVER-PASS-INCLUDES-SAME: {{^}} "-internal-externc-isystem" "{{.*}}/usr/include"

and

// DRIVER-PASS-INCLUDES-SAME: "-internal-isystem" "[[RESOURCE]]{{/|\\\\}}include"
// DRIVER-PASS-INCLUDES-SAME: "-internal-externc-isystem" "{{.*}}/usr/include"