shchenz added a comment.

Look almost good! Thanks for doing this.



================
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:
> > 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?


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