vitalybuka added subscribers: hctim, vitalybuka.
vitalybuka added a comment.
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-msan/builds/12035
-- Testing: 14692 tests, 96 threads --
Testing:
FAIL: Clang :: AST/float16.cpp (132 of 14692)
TEST 'Clang
vitalybuka added a comment.
fixed with r359366
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60485/new/
https://reviews.llvm.org/D60485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
This revision was automatically updated to reflect the committed changes.
Closed by commit rC359348: [AArch64] Add support for MTE intrinsics (authored
by javed.absar, committed by ).
Herald added a project: clang.
Changed prior to commit:
https://reviews.llvm.org/D60485?vs=196297&id=196915#toc
t.p.northover accepted this revision.
t.p.northover added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/CodeGen/CGBuiltin.cpp:7129-7131
+// Although it is possible to supply a different return
+// address (first arg) to this intrinsic,
javed.absar marked an inline comment as done.
javed.absar added inline comments.
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
t.p.northover added inline comments.
Comment at: include/clang/Sema/Sema.h:10762
bool AllowName);
+ bool SemaBuiltinARMMemoryTaggingCall(unsigned BuiltinID, CallExpr *TheCall);
public:
Slightly misaligned.
Comm
javed.absar updated this revision to Diff 196297.
javed.absar marked an inline comment as done.
javed.absar added a comment.
Hi Tim:
Have made the changes as suggested. Please have a look.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60485/new/
https://reviews.llvm.org/D60485
Files:
t.p.northover added inline comments.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:9566
+def err_memtag_arg_null_or_pointer : Error<
+ "%0 argument must be a null or a pointer (%1 invalid)">;
+def err_memtag_any2arg_pointer : Error<
javed.absar wrote:
>
javed.absar marked 2 inline comments as done.
javed.absar added inline comments.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:9566
+def err_memtag_arg_null_or_pointer : Error<
+ "%0 argument must be a null or a pointer (%1 invalid)">;
+def err_memtag_any2arg_pointer :
t.p.northover added inline comments.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:9566
+def err_memtag_arg_null_or_pointer : Error<
+ "%0 argument must be a null or a pointer (%1 invalid)">;
+def err_memtag_any2arg_pointer : Error<
I think these diagno
javed.absar updated this revision to Diff 195112.
javed.absar added a comment.
Tests merged as suggested.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60485/new/
https://reviews.llvm.org/D60485
Files:
include/clang/Basic/BuiltinsAArch64.def
include/clang/Basic/DiagnosticSemaKinds.
DavidSpickett added a comment.
Do we need a separate file for that one "__arm_mte_ptrdiff" test? Wouldn't it
be easier to wrap the whole function in "__cplusplus" and put it in the same
file.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60485/new/
https://reviews.llvm.org/D60485
_
javed.absar created this revision.
javed.absar added reviewers: DavidSpickett, olista01.
Herald added a subscriber: kristof.beyls.
This patch provides intrinsics support for Memory Tagging Extension (MTE),
which was introduced with the Armv8.5-a architecture.
These intrinsics are available whe
13 matches
Mail list logo
