nemanjai added inline comments.
================ Comment at: clang/test/CodeGen/aix-visibility-inlines-hidden.cpp:30-34 +// COMMON-ASM: mflr 0 +// COMMON-ASM-NEXT: stw 0, 8(1) +// COMMON-ASM-NEXT: stwu 1, -64(1) +// COMMON-ASM-NEXT: bl ._Z1fv +// NOP-ASM-NEXT: nop ---------------- rsmith wrote: > Generally we strongly prefer for frontend tests to test only the frontend; in > this case, that means testing only the IR that Clang is producing and not the > assembly that comes out of LLVM for that IR. This should also remove the need > to require PPC as a registered target. +1 The asm test can go into `llvm/test/CodeGen/PowerPC`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89986/new/ https://reviews.llvm.org/D89986 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
