[PATCH] D158231: [clang][test] Fix clang machine-function-split tests

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 19 14:52:57 PDT 2023


MaskRay resigned from this revision.
MaskRay added a comment.
This revision now requires review to proceed.

Sorry, after reading through D157750 <https://reviews.llvm.org/D157750>, I think the patch should be reverted.



================
Comment at: clang/test/CodeGen/fsplit-machine-functions.c:2
+// REQUIRES: system-linux
+// REQUIRES: x86-registered-target
+// REQUIRES: arm-registered-target
----------------
shenhan wrote:
> MaskRay wrote:
> > We usually place x86 tests under CodeGen/X86 and arm tests under CodeGen/ARM
> I can moved the X86 cases to clang/test/CodeGen/X86, but there is no clang/test/CodeGen/Arm or similar, shall I create "clang/test/CodeGen/Arm" instead? 
Sorry, this is not llvm/test/CodeGen. For clang/test/CodeGen this is fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158231



More information about the cfe-commits mailing list