[PATCH] D15471: [IR] Add support for floating pointer atomic loads and stores

JF Bastien via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 15 16:15:46 PST 2015


jfb accepted this revision.
jfb added a comment.
This revision is now accepted and ready to land.

lgtm after the two "integral" typo fixes.


================
Comment at: test/Verifier/atomics.ll:4
@@ +3,3 @@
+; CHECK: atomic store operand must have integer or floating point type!
+; CHECK: atomic load operand must have integer or floating point type!
+
----------------
Ha, that's not the best error message since `x86_mmx` is a floating-point type. I'll rework it in D15512 if that's OK with you.


http://reviews.llvm.org/D15471





More information about the llvm-commits mailing list