This unifies mips/mipsel and mips64/mips64el into a single class so that we can
later support O32 on mips64/mips64el and N32/N64 on mips/mipsel (when an
appropriate CPU selected).
Details
Details
- Reviewers
atanasyan - Commits
- rG4672af6fecd1: [mips] Fold MipsTargetInfoBase subclasses into MipsTargetInfoBase and rename to…
rC270984: [mips] Fold MipsTargetInfoBase subclasses into MipsTargetInfoBase and rename to…
rL270984: [mips] Fold MipsTargetInfoBase subclasses into MipsTargetInfoBase and rename…
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/Basic/Targets.cpp | ||
---|---|---|
6986 ↗ | (On Diff #58604) | What do you think about creation a static function and using it to simplify this and similar conditions? static bool is32BitTriple(const llvm::Triple &Triple) { return Triple.getArch() == llvm::Triple::mips || Triple.getArch() == llvm::Triple::mipsel; } |
7006 ↗ | (On Diff #58604) | Can we use just else here? |
lib/Basic/Targets.cpp | ||
---|---|---|
6986 ↗ | (On Diff #58604) | I'm happy to add one if you want it but I'm currently finishing a patch that removes most of them in favour of ABI checks. The only one I think should remain as an Arch check is the one for MipsTargetInfo::ABI. Even that one is incorrect on our buildbots which detect as mips64-linux-gnu but should default to O32. |
7006 ↗ | (On Diff #58604) | Sure |
Comment Actions
LGTM
lib/Basic/Targets.cpp | ||
---|---|---|
6986 ↗ | (On Diff #58604) | Agreed. In that case the separate function is redundant. |