[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...
Size: 4909 bytes
Desc: not available
More information about the cfe-commits