This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Fixing OpenMP/driver.c failing on 32-bit hosts
ClosedPublic

Authored by jhuber6 on Oct 21 2020, 11:40 AM.

Details

Summary

The changes made in D88594 caused the test OpenMP/driver.c to fail on a 32-bit host becuase it was offloading to a 64-bit architecture by default. The offloading test was moved to a new file and a feature was added to the lit config to check for a 64-bit host.

Diff Detail

Event Timeline

jhuber6 created this revision.Oct 21 2020, 11:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 21 2020, 11:40 AM
jhuber6 requested review of this revision.Oct 21 2020, 11:40 AM
jhuber6 updated this revision to Diff 299767.Oct 21 2020, 11:43 AM

Changing to just check if the target triple is of four known 64-bit architectures.

daltenty accepted this revision.Oct 21 2020, 1:56 PM

Thanks! I've verified this fixes the test on AIX.

This revision is now accepted and ready to land.Oct 21 2020, 1:56 PM
This revision was automatically updated to reflect the committed changes.