[PATCH] D105956: [AIX] Enable dollar sign as PC in inlineasm

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 13 19:23:45 PDT 2021


nemanjai added a comment.

Is it possible to also add an `MC` test case for this (similar to `test/MC/PowerPC/ppc64-operands.s`)?



================
Comment at: llvm/test/CodeGen/PowerPC/inline-asm-dollarpc.ll:3
+; RUN: llc -mcpu=pwr7 -verify-machineinstrs \
+; RUN:     -mtriple=powerpc-unknown-aix < %s  | FileCheck %s --check-prefix=IS32
+
----------------
Minor nit: `IS` isn't clear as a prefix, how about `IAS` for Integrated AS?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105956



More information about the llvm-commits mailing list