[PATCH] D24986: [MS] Implement __iso_volatile loads/stores as builtins

Martin Storsjö via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 28 02:30:43 PDT 2016

mstorsjo retitled this revision from "Headers: Add iso_volatile load/store intrinsics" to "[MS] Implement __iso_volatile loads/stores as builtins".
mstorsjo updated the summary for this revision.
mstorsjo updated this revision to Diff 72782.
mstorsjo added a comment.

Changed to implement it as builtins, as requested. I had to put this at the bottom in EmitBuiltinExpr (which returns RValues) instead of in EmitARMBuiltinExpr (which returns Value*), since the returned Value* for stores is nullptr. A nullptr return value from EmitTargetBuiltinExpr indicates that it wasn't handled, this triggered errors about the store builtins being unsupported.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24986.72782.patch
Type: text/x-patch
Size: 4909 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160928/39488e64/attachment.bin>

More information about the cfe-commits mailing list