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

Jake Egan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 8 06:19:43 PDT 2021


Jake-Egan added inline comments.


================
Comment at: clang/test/Modules/ExtDebugInfo.cpp:208
 
-
-// CHECK: !DIImportedEntity(tag: DW_TAG_imported_declaration, scope: !{{[0-9]+}}, entity: ![[STRUCT]], file: ![[CPP]], line: 27)
+// CHECK: !DIImportedEntity(tag: DW_TAG_imported_declaration, scope: !{{[0-9]+}}, entity: ![[STRUCT]], file: ![[CPP]], line: 28)
 
----------------
jsji wrote:
> Why this change? If this is needed, this should be in another patch. Thanks.
The test failed in Debian and Windows because the unsupported line I added changed the expected line number: 27->28. 
https://reviews.llvm.org/D109060?id=371137

Without this change, this patch would cause a regression. But, I can put it in another patch if that's preferred.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109060/new/

https://reviews.llvm.org/D109060



More information about the cfe-commits mailing list