This patch simply adds support for the new CPU in anticipation of
Power10. There isn't really any functionality added so there are no
associated test cases at this time.
Details
- Reviewers
stefanp nemanjai amyk hfinkel power-llvm-team - Group Reviewers
Restricted Project - Commits
- rG2368bf52cd77: [PowerPC] Add support for -mcpu=pwr10 in both clang and llvm
rG7eb666b1556b: [PowerPC] Add support for -mcpu=pwr10 in both clang and llvm
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
I believe we're also missing IsISA3_1 = false; in PPCSubtarget.cpp.
llvm/lib/Target/PowerPC/PPC.td | ||
---|---|---|
338 | Are we missing FeatureISA3_1 in P10AdditionalFeatures? |
clang/lib/Basic/Targets/PPC.cpp | ||
---|---|---|
271 | Well, pls ignore this comments as that macro should be added when p10 vector feature is added. |
clang/test/Preprocessor/init-ppc64.c | ||
---|---|---|
654 | ||
llvm/lib/Target/PowerPC/PPC.td | ||
340 | I think these can be moved up to P10AdditionalFeatures. That way everything on P10 is now inheritable by future and we don't have to specify anything for FutureSpecificFeatures. | |
351 | These features are now no longer FutureSpecificFeatures I would think that they would now be part of Power10 and should be inherited by future CPU. | |
llvm/test/CodeGen/PowerPC/check-cpu.ll | ||
11 | nit: |
clang/lib/Basic/Targets/PPC.cpp | ||
---|---|---|
335 | I think we also need to check for ArchDefinePwr10 and ArchDefineFuture based on the comment "// We have __float128 on PPC but not power 9 and above." !(ArchDefs & ArchDefinePwr9) -> !(ArchDefs & (ArchDefinePwr9 | ArchDefinePwr10 | ArchDefineFuture)) |
clang/lib/Basic/Targets/PPC.cpp | ||
---|---|---|
335 | Yeah. That makes sense. Thanks for the explanation! |
I think this looks good aside from the comments I had.
llvm/lib/Target/PowerPC/PPCSubtarget.h | ||
---|---|---|
142 | Missing IsISA3_1 = false; in PPCSubtarget.cpp. |
Do we miss to define this macro ?