MaskRay added inline comments.
================
Comment at: clang/test/Driver/target-as.s:1
 // Make sure the -march is passed down to cc1as.
 // RUN: %clang -target i386-unknown-freebsd -### -c -integrated-as %s \
----------------
shchenz wrote:
> MaskRay wrote:
> > aix-as.c will be more appropriate.
> Do you mean we change this file to `aix-as.s` or we change back to previous 
> version? The file `target-as.s` can not be renamed to `aix-as.s`, it also 
> contains tests for i386, see the first run line.
You can add the new tests to the existing `aix-as.s`. The behavior is about 
assemblers on AIX. `aix-as.s` seems more appropriate than `target-as.s`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148490

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to