[PATCH] D136612: [lit][REQUIRES] Fix some tests with incorrect REQUIRES clauses

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 25 00:55:37 PDT 2022


jhenderson added inline comments.


================
Comment at: llvm/test/Analysis/BasicAA/cs-cs-arm.ll:3-4
+; REQUIRES: arm-registered-target
+; This hasn't been run in a long time and it no longer matches reality.
+; XFAIL: *
 
----------------
Have you raised a bug for this?


================
Comment at: llvm/test/CodeGen/Hexagon/abi-padding-2.ll:2
 ; RUN: llc -march=hexagon -mcpu=hexagonv65 < %s | FileCheck %s
-; REQUIRES: hexagon
 
----------------
I'm not familiar with these tests, so this might be a silly question, but why isn't this required? Is it in the lit config?


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

https://reviews.llvm.org/D136612



More information about the llvm-commits mailing list