compnerd added a comment.

Could you please add a test to ensure that Darwin defaults to the old behaviour?



================
Comment at: include/clang/Basic/LangOptions.h:142
+    /// Attempt to be ABI-compatible with code generated by Clang 8.0.x
+    /// (https://reviews.llvm.org/D59744). This causes __m64 to be passed in
+    /// MMX register instead of integer register.
----------------
Referencing the SVN revision is better IMO as the infrastructure can change, 
but, the reference to the commit is useful as that can be mapped.


Repository:
  rC Clang

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

https://reviews.llvm.org/D63473



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

Reply via email to