This revision was automatically updated to reflect the committed changes.
Closed by commit rL370716: [ARM NEON] Avoid duplicated decarations (authored by
dnsampaio, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.ll
dnsampaio updated this revision to Diff 218412.
dnsampaio added a comment.
- Fix comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66588/new/
https://reviews.llvm.org/D66588
Files:
clang/utils/TableGen/NeonEmitter.cpp
Index: clang/utils/Ta
ostannard added a comment.
LGTM with one small nit.
Comment at: clang/utils/TableGen/NeonEmitter.cpp:1906
std::string Intrinsic::generate() {
+ // Avoid duplicated code for big and small endians
+ if (isBigEndianSafe()) {
s/small endians/little endian/
Rep
dnsampaio updated this revision to Diff 217594.
dnsampaio added a comment.
Fix / Update / Rebase
- Avoid appending __noswap_ to intrinsics that are BigEndianSafe
- Moved to monorepo
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66588/new/
https://
dnsampaio planned changes to this revision.
dnsampaio added a comment.
Breaks the header. Needs to avoid generating calls to functions with predicated
__noswap when it is BigEndianSafe.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66588/new/
https://reviews.llv
dnsampaio updated this revision to Diff 216801.
dnsampaio added a comment.
- Consider BigEndianSafe intrinsics that all inputs and outputs are scalar or
single element vectors
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66588/new/
https://reviews.llvm.org/D665
dnsampaio created this revision.
dnsampaio added reviewers: t.p.northover, ostannard.
Herald added subscribers: cfe-commits, kristof.beyls, javed.absar.
Herald added a project: clang.
The declaration of arm neon intrinsics that are
"big endian safe" print the same code for big
and small endian tar