This revision was automatically updated to reflect the committed changes.
Closed by commit rG96d4ccf00c8f: [VE] Clang toolchain for VE (authored by kaz7,
committed by simoll).
Changed prior to commit:
https://reviews.llvm.org/D79411?vs=272632&id=272932#toc
Repository:
rG LLVM Github Monorepo
kaz7 added a comment.
Please check an inline comment.
Comment at: clang/lib/Basic/Targets/VE.h:166
+
+ int getEHDataRegisterNumber(unsigned RegNo) const override {
+// S0 = ExceptionPointerRegister, S1 = ExceptionSelectorRegister
Current VE backend doesn't
kaz7 accepted this revision.
kaz7 added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
___
cf
simoll updated this revision to Diff 272632.
simoll marked an inline comment as done.
simoll added a comment.
Removed `getEHDataRegisterNumber`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
Files:
clan
kaz7 added reviewers: hfinkel, b-sumner, jmolloy, kparzysz, majnemer, igorb.
kaz7 added a comment.
Add people who reviewed patches adding new target to clang. Sorry for
bothering you guys, but I appreciate if you review our patch or give us
suggestions how to get reviewed. Thanks in advance.
kaz7 added a comment.
It looks fine to me but I'm not clang expert. Can anyone review this or
introduce us reviewers? Thanks.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
___
simoll added a comment.
Ping. Anything i can do to get this reviewed?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
___
cfe-commits mailing list
cfe-commits@l
simoll added a comment.
Leaving a note: https://reviews.llvm.org/D79084 depends on this patch to build
compiler-rt.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
___
simoll updated this revision to Diff 263638.
simoll added a comment.
- Simplified: stripped down to a basic C GNU Linux toolchain hard-float, no C++
stdlib.
- rebased.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.o
simoll added a comment.
FYI This passes all tests on the mergebot. There are some obscure lint
suggestions that make it report a failure.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79411/new/
https://reviews.llvm.org/D79411
_
simoll created this revision.
simoll added reviewers: arsenm, k-ishizaka, kaz7.
simoll added a project: VE.
Herald added subscribers: cfe-commits, jfb, krytarowski, mgorny, wdng.
Herald added a project: clang.
This patch enables compilation of C code for the VE target with Clang.
Repository:
r
11 matches
Mail list logo