[PATCH] D60485: [AArch64] Add support for MTE intrinsics
Tim Northover via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 24 03:14:00 PDT 2019
t.p.northover added inline comments.
Comment at: include/clang/Sema/Sema.h:10762
+ bool SemaBuiltinARMMemoryTaggingCall(unsigned BuiltinID, CallExpr *TheCall);
Comment at: lib/CodeGen/CGBuiltin.cpp:7129-7131
+ // Although it is possible to supply a different return
+ // address (first arg) to this intrinsic, for now we set
+ // return address same as input address.
I think this should be fixed now. It looks like technical debt from the fact that the instructions only fairly recently gained that feature after the intrinsics were implemented internally. There's no good way to justify the current semantics to someone unaware of that history.
CHANGES SINCE LAST ACTION
More information about the cfe-commits