george.burgess.iv added a comment.

I like the idea of fixing those things, too; I'll start poking them soon. :)

Even if we do end up fixing all of that, I still think it would be good to try 
to diagnose this in the frontend. So, if anyone has comments on this while I'm 
staring at the aarch64 backend, please let me know.



================
Comment at: test/CodeGen/aarch64-mgeneral_regs_only.c:2
+// RUN: %clang -target aarch64 -mgeneral-regs-only %s -o - -S -emit-llvm | 
FileCheck %s
+// %clang -target aarch64 -mgeneral-regs-only %s -o - -S | FileCheck %s 
--check-prefix=ASM
+
----------------
Oops, forgot a `RUN:` here.


https://reviews.llvm.org/D38479



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

Reply via email to