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 this? -emit-obj is already excluded?
> > Excluding this test causes regressions because LIT can't find any tests at 
> > `/codegen-body` directory:
> > ```
> > Clang :: utils/update_cc_test_checks/global-hex-value-regex.test
> > Clang :: utils/update_cc_test_checks/global-value-regex.test
> > ```
> > I figure it's easier to mark this test unsupported. 
> Is it possible to add `expected-no-diagnostics` for any of the above two RUN 
> lines? So we will have a test point and be able to exclude this case?
Unfortunately adding `expected-no-diagnostics` fix the issue. The existing 
`expected-no-diagnostics` is already applied to all the RUN lines.


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
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to