[PATCH] D151938: [clang][index] NFCI: Make `CXFile` a `FileEntryRef`

Jake-Egan added a comment. @jansvoboda11 Actually, we could give you access to an AIX machine to debug this. Would that work for you? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151938/new/ https://reviews.llvm.org/D151938

[PATCH] D151938: [clang][index] NFCI: Make `CXFile` a `FileEntryRef`

Jake-Egan added a comment. @jansvoboda11 Thanks for taking care of the other tests. I'm not sure how this could be reproduced not on AIX. I'll have to investigate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151938/new/

[PATCH] D159213: [AIX][ClangRepl] Disable new tests on AIX

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG86337beca2e6: [AIX][ClangRepl] Disable new tests on AIX (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D159213: [AIX][ClangRepl] Disable new tests on AIX

Jake-Egan created this revision. Herald added a project: All. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D159213 Files:

[PATCH] D151938: [clang][index] NFCI: Make `CXFile` a `FileEntryRef`

Jake-Egan added a comment. Sorry for the late response @jansvoboda11, when `lookupModuleFile` is called, the file `build/tools/clang/test/Index/Core/Output/index-with-module.m.tmp.mcp/A03A61VI43WA/ModA-21USRMHJNU3PG.pcm` doesn't exist. So it seems there's some issue with the creation of the

[PATCH] D153600: Implement -frecord-command-line for XCOFF

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbbd0d123d3aa: Implement -frecord-command-line for XCOFF (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan added inline comments. Comment at: llvm/lib/MC/MCAsmStreamer.cpp:981 + + // Metadata needs to be padded out to an even word size. + size_t MetadataSize = Metadata.size(); scott.linder wrote: > stephenpeckham wrote: > > There's no requirement to pad

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 537572. Jake-Egan added a comment. Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600 Files: clang/lib/Driver/ToolChains/Clang.cpp

[PATCH] D143241: [Clang] Reset FP options before function instantiations

Jake-Egan added a comment. Hi, the following assertion is getting tripped on AIX: Assertion failed: *FpPragmaCurrentValue == SemaObj->FpPragmaStack.DefaultValue && "Expected a default pragma float_control value", file

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 537377. Jake-Egan added a comment. Rerun CI Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600 Files: clang/lib/Driver/ToolChains/Clang.cpp

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 537160. Jake-Egan added a comment. Fix formatting CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600 Files: clang/lib/Driver/ToolChains/Clang.cpp llvm/include/llvm/CodeGen/AsmPrinter.h

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 537156. Jake-Egan edited the summary of this revision. Jake-Egan added a comment. Updated to use both a newline and null byte to separate command lines. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 536920. Jake-Egan edited the summary of this revision. Jake-Egan added a comment. Thanks for the review @stephenpeckham, I updated the patch with the requested changes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 536890. Jake-Egan added a comment. Fix formatting CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600 Files: clang/lib/Driver/ToolChains/Clang.cpp llvm/include/llvm/CodeGen/AsmPrinter.h

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan marked an inline comment as done. Jake-Egan added inline comments. Comment at: llvm/lib/MC/MCAsmStreamer.cpp:986 + + // If there's no metadata, the length is 0. + if (MetadataSize == 0) { Handled the 0 length case here. CHANGES SINCE LAST ACTION

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan marked 6 inline comments as done. Jake-Egan added inline comments. Comment at: llvm/lib/MC/MCAsmStreamer.cpp:1022 + if (PaddingSize) { +assert(PaddedSize - Index == WordSize); +std::array LastWord = {0}; I changed the assert that you requested

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan updated this revision to Diff 536795. Jake-Egan added a comment. Thanks for the review @scott.linder. I applied the changes you requested with some differences. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153600/new/ https://reviews.llvm.org/D153600 Files:

[PATCH] D153600: Implement -frecord-command-line for XCOFF

Jake-Egan created this revision. Herald added subscribers: kbarton, hiraditya, nemanjai. Herald added a project: All. Jake-Egan requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, MaskRay. Herald added projects: clang, LLVM. Repository: rG LLVM Github

[PATCH] D151938: [clang][index] NFCI: Make `CXFile` a `FileEntryRef`

Jake-Egan added a comment. Hi, there's a test failure on AIX, could you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/8005/steps/6/logs/FAIL__Clang__index-with-module_m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D150282: [Driver] -ftime-trace: derive trace file names from -o and -dumpdir

Jake-Egan added a comment. Hi, this fails on AIX. It looks like it still produces the `/tmp/lit-tmp-*/[ab]-*.json` format. Can you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/7429/steps/6/logs/FAIL__Clang__ftime-trace_cpp Repository: rG LLVM Github Monorepo

[PATCH] D145600: [AIX] change "llvm-ar" to "env OBJECT_MODE=any llvm-ar" in clang/test for AIX OS

Jake-Egan accepted this revision. Jake-Egan added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145600/new/ https://reviews.llvm.org/D145600

[PATCH] D145098: [clang][deps] Preserve input ordering in the full output

Jake-Egan added a comment. Thanks for addressing this. Unfortunately the test is still failing after the fix and a clean build https://lab.llvm.org/buildbot/#/builders/214/builds/6170/steps/6/logs/FAIL__Clang__modules-full-output-tu-order_c Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D145098: [clang][deps] Preserve input ordering in the full output

Jake-Egan added a comment. Hi, this new test fails on AIX, could you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/6148/steps/6/logs/FAIL__Clang__modules-full-output-tu-order_c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D142560: Allow getRawCommentForDecl to find comments in macros

Jake-Egan added a comment. Thank you for looking into the failure. Fortunately the test is passing on the AIX bot now and locally for me - it looks like a clean build fixed it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142560/new/

[PATCH] D142560: Allow getRawCommentForDecl to find comments in macros

Jake-Egan added a comment. Hi, this change causes a failure on AIX. Could you take a look? https://lab.llvm.org/buildbot/#/builders/214/builds/5502/steps/6/logs/FAIL__Clang__annotate-comments-objc_m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D142484: [LinkerWrapper] Only import static libraries with needed symbols

Jake-Egan added a comment. In D142484#4079909 , @jhuber6 wrote: > In D142484#4079869 , @Jake-Egan > wrote: > >> Hi, this new test fails on AIX. Could you take a look? >>

[PATCH] D142484: [LinkerWrapper] Only import static libraries with needed symbols

Jake-Egan added a comment. Hi, this new test fails on AIX. Could you take a look? https://lab.llvm.org/buildbot/#/builders/214/builds/5477/steps/6/logs/FAIL__Clang__linker-wrapper-libs_c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142484/new/

[PATCH] D137058: [Driver] [Modules] Support -fmodule-output (1/2)

Jake-Egan added a comment. In D137058#4057424 , @ChuanqiXu wrote: > In D137058#4056647 , @Jake-Egan > wrote: > >> Hi, this new test fails on AIX >>

[PATCH] D137058: [Driver] [Modules] Support -fmodule-output (1/2)

Jake-Egan added a comment. Hi, this new test fails on AIX https://lab.llvm.org/buildbot/#/builders/214/builds/5351/steps/6/logs/FAIL__Clang__module-output_cppm Could you take a look? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137058/new/

[PATCH] D141440: [OpenMP] Adjust phases for AMDGPU offloading for OpenMP in save-temps mode

Jake-Egan added a comment. Getting a build failure on AIX, could you take a look please? /home/powerllvm/powerllvm_env/aix-ppc64/clang-ppc64-aix/llvm-project/clang/lib/Driver/Driver.cpp:5715:37: error: lambda capture 'C' is not used [-Werror,-Wunused-lambda-capture] auto

[PATCH] D140867: [C++20] [Modules] Don't generate global ctors/dtors for variables which are available externally

Jake-Egan added a comment. In D140867#4042405 , @ChuanqiXu wrote: > In D140867#4042351 , @Jake-Egan > wrote: > >>> Would you like to take a double look? >> >> Yes, it still fails on the bot and on my local

[PATCH] D140867: [C++20] [Modules] Don't generate global ctors/dtors for variables which are available externally

Jake-Egan added a comment. > Would you like to take a double look? Yes, it still fails on the bot and on my local machine. I've added an XFAIL to the test for now just to get the AIX bot green. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D140867: [C++20] [Modules] Don't generate global ctors/dtors for variables which are available externally

Jake-Egan added a comment. In D140867#4038388 , @ChuanqiXu wrote: > In D140867#4036239 , @Jake-Egan > wrote: > >> Hi, this test is failing on AIX, could you take a look please? >>

[PATCH] D140867: [C++20] [Modules] Don't generate global ctors/dtors for variables which are available externally

Jake-Egan added a comment. Hi, this test is failing on AIX, could you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/5242/steps/6/logs/FAIL__Clang__pr59765-modules-global-ctor-dtor_cppm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D130689: [LLVM] Update C++ standard to 17

Jake-Egan added a comment. There is a failure on the AIX bot also: 152.827 [4302/10/270] Linking CXX executable bin/llvm-tblgen FAILED: bin/llvm-tblgen : && /opt/IBM/openxlC/17.1.0/bin/ibm-clang++_r -mcmodel=large -fPIC -Werror -Werror=date-time -Werror=unguarded-availability-new -Wall

[PATCH] D130581: [clang][ExtractAPI] Ensure that class properties have a kind of "Type Property"

Jake-Egan added a comment. Hi, this new test is failing on AIX, could you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/2544/steps/6/logs/FAIL__Clang__objc_property_m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D121997: [clang][driver] Fix compilation database dump with multiple architectures

Jake-Egan added a comment. Hi, it seems on AIX the target and arch options are ignored, so we don't get the expected output for this test. Any ideas about the course of action we should take?

[PATCH] D125847: LTO: Add option to initialize with opaque/non-opaque pointer types

Jake-Egan added a comment. Thanks for taking a look. I agree someone on AIX should investigate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125847/new/ https://reviews.llvm.org/D125847 ___ cfe-commits

[PATCH] D125847: LTO: Add option to initialize with opaque/non-opaque pointer types

Jake-Egan added a comment. In D125847#3554706 , @MatzeB wrote: > In D125847#3554569 , @Jake-Egan > wrote: > >> Hi, this caused `arm-float-abi-lto.c` to fail on AIX. The failure went away >> for a few builds,

[PATCH] D125847: LTO: Add option to initialize with opaque/non-opaque pointer types

Jake-Egan added a comment. Hi, this caused `arm-float-abi-lto.c` to fail on AIX. The failure went away for a few builds, then came back. Could you take a look? https://lab.llvm.org/buildbot/#/builders/214/builds/1625/steps/6/logs/FAIL__Clang__arm-float-abi-lto_c Repository: rG LLVM Github

[PATCH] D126031: [libclang] add supporting for indexing/visiting C++ concepts

Jake-Egan added a comment. @arphaman Thanks for the quick action. The tests pass on a clean build https://lab.llvm.org/buildbot/#/builders/214/builds/1493, so that's probably why there was trouble reproducing. I think we could revert 1b34f1e996565bc5e4f2be14b89f881f8fe0f3b9

[PATCH] D126031: [libclang] add supporting for indexing/visiting C++ concepts

Jake-Egan added a comment. Hi, these tests fail on AIX, could you take a look please? https://lab.llvm.org/buildbot/#/builders/214/builds/1489 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126031/new/ https://reviews.llvm.org/D126031

[PATCH] D124534: [clang] Add a diagnostic for line directive of a gnu extension

Jake-Egan added a comment. Hi, this patch causes a test failure on AIX https://lab.llvm.org/buildbot/#/builders/214/builds/1221/steps/6/logs/FAIL__Clang__noinline_cu Could you please make a fix or revert if it takes too long? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D123273: [utils] Avoid hardcoding metadata ids in update_cc_test_checks

Jake-Egan added a comment. 4 of the tests fail on AIX also: https://lab.llvm.org/buildbot/#/builders/214/builds/1207 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123273/new/ https://reviews.llvm.org/D123273

[PATCH] D124250: [Serialization] write expr dependence bits as a single integer

Jake-Egan added a comment. Thanks @sammccall, the test is passing now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124250/new/ https://reviews.llvm.org/D124250 ___ cfe-commits mailing list

[PATCH] D124250: [Serialization] write expr dependence bits as a single integer

Jake-Egan added a comment. Hi, this broke a test on AIX https://lab.llvm.org/buildbot/#/builders/214/builds/903/steps/6/logs/FAIL__Clang__pch-with-module_m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124250/new/ https://reviews.llvm.org/D124250

[PATCH] D122983: [C11/C2x] Change the behavior of the implicit function declaration warning

Jake-Egan added a comment. @aaron.ballman Here are all the errors. Thanks! MultiSource/Applications/d/write_ctables.c:600:6: error: call to undeclared library function 'strncasecmp' with type 'int (const char *, const char *, unsigned long)'; ISO C99 and later do not support implicit

[PATCH] D122983: [C11/C2x] Change the behavior of the implicit function declaration warning

Jake-Egan added a comment. Hi, we also have a failure on AIX with test-suite `call to undeclared library function '%0' with type %1; ISO C99 and later` https://lab.llvm.org/buildbot/#/builders/214/builds/825/steps/9/logs/stdio Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D123345: Treat `std::move`, `forward`, and `move_if_noexcept` as builtins.

Jake-Egan added a comment. Hi, unfortunately there's a build failure on AIX: https://lab.llvm.org/buildbot/#/builders/214/builds/779/steps/9/logs/stdio. Could you take a look? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123345/new/

[PATCH] D122519: [NFC][tests][AIX] XFAIL test for lack of visibility support

This revision was automatically updated to reflect the committed changes. Closed by commit rGf5a9b5cc1265: [NFC][tests][AIX] XFAIL test for lack of visibility support (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122519/new/

[PATCH] D122519: [AIX] XFAIL test for lack of visibility support

Jake-Egan updated this revision to Diff 418369. Jake-Egan added a comment. Updated the comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122519/new/ https://reviews.llvm.org/D122519 Files: clang/test/OpenMP/target_update_messages.cpp

[PATCH] D122519: [AIX] XFAIL test for lack of visibility support

Jake-Egan created this revision. Herald added a project: All. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D122519 Files:

[PATCH] D121709: [NFC][AIX] Disable precompiled module file test on AIX

This revision was automatically updated to reflect the committed changes. Closed by commit rGd1873d395d57: [NFC][AIX] Disable precompiled module file test on AIX (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121709/new/

[PATCH] D121709: [NFC][AIX] Disable precompiled module file test on AIX

Jake-Egan created this revision. Herald added a project: All. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D121709 Files:

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6f4f745668ec: [clang][deps] Adapt test to be compatible when the assembler is called by… (authored by Jake-Egan). Repository: rG LLVM Github

[PATCH] D118477: [NFC][AIX]Disable new pcm tests on AIX

Jake-Egan accepted this revision. Jake-Egan added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118477/new/ https://reviews.llvm.org/D118477

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Jake-Egan updated this revision to Diff 403763. Jake-Egan added a comment. Fixed diff. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118152/new/ https://reviews.llvm.org/D118152 Files: clang/test/ClangScanDeps/headerwithdirname.cpp

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Jake-Egan updated this revision to Diff 403761. Jake-Egan added a comment. Updated patch to use a more accurate check. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118152/new/ https://reviews.llvm.org/D118152 Files:

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Jake-Egan added inline comments. Comment at: clang/test/ClangScanDeps/headerwithdirname.cpp:16 -// CHECK: headerwithdirname_input.o +// CHECK: headerwithdirname_input{{.*}}{{.o|.s}} // CHECK-NEXT: headerwithdirname_input.cpp jansvoboda11 wrote: > Why is

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D118152 Files: clang/test/ClangScanDeps/headerwithdirname.cpp

[PATCH] D117587: [ifs] Use a tmp file instead of "-"

This revision was automatically updated to reflect the committed changes. Closed by commit rGf24fe96f469b: [ifs] Use a tmp file instead of - (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117587/new/

[PATCH] D117587: [ifs] Use a tmp file instead of "-"

Jake-Egan added inline comments. Comment at: clang/test/InterfaceStubs/object.c:2 // RUN: %clang_cc1 -fvisibility default -o - -emit-interface-stubs %s | FileCheck -check-prefix=CHECK-TAPI %s -// RUN: %clang -fvisibility=default -c -o - %s | llvm-nm - 2>&1 | FileCheck

[PATCH] D117587: [ifs] Use a tmp file instead of "-"

Jake-Egan updated this revision to Diff 401254. Jake-Egan marked an inline comment as done. Jake-Egan added a comment. Removed the unncessary pipe. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117587/new/ https://reviews.llvm.org/D117587 Files:

[PATCH] D117587: [ifs] Use a tmp file instead of -

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D117587 Files: clang/test/InterfaceStubs/object.c Index:

[PATCH] D117315: [AIX][ZOS] Handle unsupported builtin function CFStringMakeConstantString

This revision was automatically updated to reflect the committed changes. Closed by commit rG6f0977519d12: [AIX][ZOS] Handle unsupported builtin function CFStringMakeConstantString (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D117315: [AIX][ZOS] Handle unsupported builtin function CFStringMakeConstantString

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D117315 Files: clang/lib/Sema/SemaChecking.cpp

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGfbf52caf58b3: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support (authored by Jake-Egan). Repository: rG LLVM Github

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added inline comments. Comment at: clang/test/ASTMerge/codegen-body/test.c:1 +// UNSUPPORTED: powerpc64-ibm-aix // RUN: %clang_cc1 -emit-pch -o %t.1.ast %S/Inputs/body1.c Jake-Egan wrote: > shchenz wrote: > > Jake-Egan wrote: > > > shchenz wrote: > >

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added inline comments. Comment at: clang/test/ASTMerge/codegen-body/test.c:1 +// UNSUPPORTED: powerpc64-ibm-aix // RUN: %clang_cc1 -emit-pch -o %t.1.ast %S/Inputs/body1.c shchenz wrote: > Jake-Egan wrote: > > shchenz wrote: > > > Do we still need

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan marked an inline comment as done. Jake-Egan added inline comments. Comment at: clang/test/ASTMerge/codegen-body/test.c:1 +// UNSUPPORTED: powerpc64-ibm-aix // RUN: %clang_cc1 -emit-pch -o %t.1.ast %S/Inputs/body1.c shchenz wrote: > Do we still need

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 395904. Jake-Egan added a comment. Fixed typo. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 Files: clang/test/ASTMerge/codegen-body/test.c

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan marked an inline comment as done. Jake-Egan added inline comments. Comment at: clang/test/ClangScanDeps/modules-full-by-mod-name.cpp:1 +// UNSUPPORTED: powerpc64-ibm-aix // RUN: rm -rf %t.dir shchenz wrote: > Jake-Egan wrote: > > jsji wrote: > > >

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 395724. Jake-Egan added a comment. Added `-fintegrated-as` option. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 Files:

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D114484: [NFC][AIX]Disable unsupported hip test on AIX

Jake-Egan added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114484/new/ https://reviews.llvm.org/D114484 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D114381: [AIX] Disable unsupported offloading gpu tests

This revision was automatically updated to reflect the committed changes. Closed by commit rG0796869e4e1e: [AIX] Disable unsupported offloading gpu tests (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114381/new/

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 389849. Jake-Egan added a comment. Removed some tests already disabled by D114481 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added inline comments. Comment at: clang/test/ClangScanDeps/modules-full-by-mod-name.cpp:1 +// UNSUPPORTED: powerpc64-ibm-aix // RUN: rm -rf %t.dir jsji wrote: > there is no `fmodule-format=obj` here, why are we failing here? > They have the same

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 389196. Jake-Egan added a comment. Thanks for the review. I updated the patch to use lit.cfg.py to filter tests that use obj options. For tests that don't use the option or has an individual folder, I changed them to UNSUPPORTED instead of XFAIL.

[PATCH] D114381: [AIX] Disable unsupported offloading gpu tests

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D114381 Files: clang/test/Driver/clang-offload-bundler-asserts-on.c

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan added a comment. In D113049#3140160 , @jsji wrote: > Can we use `UNSUPPORTED` instead of `XFAIL` since it is unsupported? If 64-bit XCOFF object files will be supported in the future, I think it makes more sense to use `XFAIL` because these

[PATCH] D113428: [AIX] Define WCHAR_T_TYPE as unsigned int on 64-bit AIX for wchar.c test

This revision was automatically updated to reflect the committed changes. Closed by commit rG9303c7da3967: [AIX] Define WCHAR_T_TYPE as unsigned int on 64-bit AIX for wchar.c test (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D113614: Disable clang-repl tests failing due to lack of 64-bit XCOFF support.

Jake-Egan accepted this revision. Jake-Egan added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113614/new/ https://reviews.llvm.org/D113614

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 385830. Jake-Egan added a comment. Add `LLVM :: DebugInfo/attr-btf_type_tag.ll` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 Files:

[PATCH] D113428: [AIX] Define WCHAR_T_TYPE as unsigned int on 64-bit AIX for wchar.c test

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D113428 Files: clang/test/Sema/wchar.c Index: clang/test/Sema/wchar.c

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 384521. Jake-Egan added a comment. Updated patch to XFAIL only on 64-bit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 Files:

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan updated this revision to Diff 384220. Jake-Egan added a comment. Fix llvm/test/DebugInfo/omit-empty.ll. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113049/new/ https://reviews.llvm.org/D113049 Files:

[PATCH] D113049: [AIX] Disable tests that fail because of no 64-bit XCOFF object file support

Jake-Egan created this revision. Herald added subscribers: ormris, lxfind, pengfei, arphaman, steven_wu, hiraditya. Jake-Egan requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo

[PATCH] D112390: [AIX][ZOS] Disable tests due to lack of Objective-C support

This revision was automatically updated to reflect the committed changes. Closed by commit rG1ff1bcab970a: [AIX][ZOS] Disable tests due to lack of Objective-C support (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112390/new/

[PATCH] D112390: [AIX][ZOS] Disable tests due to lack of Objective-C support

Jake-Egan updated this revision to Diff 381979. Jake-Egan added a comment. Fixed objc_direct.ll Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112390/new/ https://reviews.llvm.org/D112390 Files: clang/test/Modules/ModuleDebugInfo.cpp

[PATCH] D112390: [AIX][ZOS] Disable module tests due to lack of Objective-C support

Jake-Egan updated this revision to Diff 381814. Jake-Egan edited the summary of this revision. Jake-Egan added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Add llvm/test/DebugInfo/X86/objc_direct.ll Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D112390: [AIX][ZOS] Disable module tests due to lack of Objective-C support

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D112390 Files: clang/test/Modules/ModuleDebugInfo.cpp

[PATCH] D111169: [AIX] Disable tests failing due to assert on DWARF object writing

Jake-Egan abandoned this revision. Jake-Egan added a comment. Abandoning this patch because D97184 is already committed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69/new/ https://reviews.llvm.org/D69

[PATCH] D111336: [AIX] Disable tests failing due to missing DWARF sections

This revision was automatically updated to reflect the committed changes. Closed by commit rG8037481cb2d7: [AIX] Disable tests failing due to missing DWARF sections (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111336/new/

[PATCH] D111336: [AIX] Disable tests failing due to missing DWARF sections

Jake-Egan created this revision. Herald added a subscriber: arphaman. Jake-Egan requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D111336 Files:

[PATCH] D111169: [AIX] Disable tests failing due to assert on DWARF object writing

Jake-Egan added a comment. In D69#3047761 , @shchenz wrote: > Thanks for working on this. > > Could you please help to rerun the affected cases after applying D97184 > ? That patch is approved and I will deliver >

[PATCH] D111169: [AIX] Disable tests failing due to assert on DWARF object writing

Jake-Egan updated this revision to Diff 377306. Jake-Egan added a comment. Updated to preserve blank lines. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69/new/ https://reviews.llvm.org/D69 Files:

[PATCH] D111169: [AIX] Disable tests failing due to assert on DWARF object writing

Jake-Egan created this revision. Herald added subscribers: ormris, pengfei, arphaman, steven_wu, hiraditya, emaste. Jake-Egan requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, MaskRay. Herald added projects: clang, LLVM. Repository: rG LLVM Github Monorepo

[PATCH] D109060: [AIX][ZOS] Disable LIT tests on AIX and z/OS due to lack of Objective-C support

This revision was automatically updated to reflect the committed changes. Closed by commit rG22f01cd4fc5d: [AIX][ZOS] Disable LIT tests on AIX and z/OS due to lack of Objective-C support (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

  1   2   >