majnemer added inline comments.
> mstorsjo wrote in ms-volatile-arm.c:2 > Well, originally, the point was to clarify that these volatile stores end up > without atomic semantics, regardless of whether -volatile:ms has been > specified. The original version of this patch (with an inline implementation > in intrin.h, just using a normal volatile pointer) wouldn't pass this test, > but would pass if I remove this option. But if you prefer, I can leave it out. Makes sense. https://reviews.llvm.org/D24986 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
