This revision was automatically updated to reflect the committed changes.
Closed by commit rC333157: [Sparc] Use the leon arch for Leon3s when
using an external assembler (authored by dcederman, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D47138
Files:
dcederman updated this revision to Diff 148170.
dcederman added a comment.
Added missing tests.
https://reviews.llvm.org/D47138
Files:
lib/Driver/ToolChains/Arch/Sparc.cpp
test/Driver/sparc-as.c
Index: test/Driver/sparc-as.c
joerg added a comment.
This still needs a test case?
https://reviews.llvm.org/D47138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyknight accepted this revision.
jyknight added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D47138#1107637, @dcederman wrote:
> I did not find a good way to access the SparcCPUInfo struct from here. No
> other arch under Toolchains seems to access
dcederman added a comment.
In https://reviews.llvm.org/D47138#1106509, @jyknight wrote:
> Separately, I think it'd be a good idea to refactor to put this info into the
> SparcCPUInfo struct, so that it's harder for them to get out of sync.
I did not find a good way to access the SparcCPUInfo
dcederman updated this revision to Diff 147965.
dcederman added a comment.
Also use the leon arch for the Myriad processors.
https://reviews.llvm.org/D47138
Files:
lib/Driver/ToolChains/Arch/Sparc.cpp
Index: lib/Driver/ToolChains/Arch/Sparc.cpp
jyknight added a comment.
There are more leon-based v8 CPUs listed in clang/lib/Basic/Targets/Sparc.cpp,
which need to be listed here, also.
Separately, I think it'd be a good idea to refactor to put this info into the
SparcCPUInfo struct, so that it's harder for them to get out of sync.
dcederman created this revision.
dcederman added a reviewer: jyknight.
Herald added subscribers: cfe-commits, jrtc27, fedor.sergeev.
This allows the use of the casa instruction available in most Leon3's.
Repository:
rC Clang
https://reviews.llvm.org/D47138
Files: