sdardis accepted this revision. sdardis added a comment. This revision is now accepted and ready to land.
LGTM. ================ Comment at: test/Driver/mips-longcalls-warning.c:1 +// REQUIRES: mips-registered-target +// RUN: %clang -### -c -target mips-mti-elf -mlong-calls %s 2>&1 | FileCheck -check-prefix=IMPLICIT %s ---------------- atanasyan wrote: > sdardis wrote: > > Can you put this in test/Driver/mips-abicalls-warning.c from the other > > patch? I'd prefer to keep all the abicalls warnings together rather than > > separate tests based on the feature the conflict with. > > Can you put this in test/Driver/mips-abicalls-warning.c from the other > > patch? > > I can, but in that case it's probably better to join both patches because > that patch loses its test. > You can do it as an NFC change afterward both commits in that case. Repository: rL LLVM https://reviews.llvm.org/D36551 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
