This is an archive of the discontinued LLVM Phabricator instance.

[NFC] [llvm-ar] Change use of empty to fix test on AIX
AbandonedPublic

Authored by qiucf on Aug 3 2021, 2:13 AM.

Details

Summary

This is a workaround for building LLVM/Clang using XL compiler on AIX.

Due to internal bug in the compiler, Filter here will be wrongly evaluated in later if statement. Several tests are failing:

LLVM :: Object/archive-extract.test
LLVM :: Object/check_binary_output.ll
LLVM :: tools/llvm-ar/full-path-option.test
LLVM :: tools/llvm-ar/count.test
LLVM :: tools/llvm-ar/print.test
LLVM :: tools/llvm-ar/extract.test
LLVM :: tools/llvm-ar/absolute-paths.test
LLVM :: tools/llvm-ar/path-names.test
LLVM :: tools/llvm-ar/response-utf8.test

We're investigating about the reason. Currently this fix will make it work properly.

Diff Detail

Event Timeline

qiucf created this revision.Aug 3 2021, 2:13 AM
qiucf requested review of this revision.Aug 3 2021, 2:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2021, 2:13 AM
jsji accepted this revision.EditedAug 3 2021, 6:43 AM

Please add comments for the workaround, so that we can find and revert once problem in build compiler is fixed.

This revision is now accepted and ready to land.Aug 3 2021, 6:43 AM
jsji added a comment.Aug 3 2021, 8:46 AM

Please add comments for the workaround, so that we can find and revert once problem in build compiler is fixed.

Actually, now we have a fix in build compiler already, so we might not need this workaround any more.

Actually, now we have a fix in build compiler already, so we might not need this workaround any more.

We will need to deploy the fix to the external-facing buildbots .

qiucf abandoned this revision.Aug 3 2021, 10:36 PM

Thanks. Now let's abandon this.