chrib added inline comments.

================
Comment at: test/CodeGen/arm-fp-eabi.c:1
+// RUN: %clang %s -target arm-none-eabi -O2 -S -o nofp.s
+// RUN: not grep '.setfp' nofp.s
----------------
ahatanak wrote:
> Instead of generating a .s file and looking for ".setfp", you can pass -### 
> and check the presence or absence of -mdisable-fp-elim (see other tests in 
> test/Driver).
yes sure, I extended the existing frame-pointer.c with the test similar to 
other archs. Thanks


https://reviews.llvm.org/D31972



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

Reply via email to