Skip to content

Commit 59181e4

Browse files
committed
[CIR][CIRGen] Change code style
1 parent 1ad5677 commit 59181e4

File tree

2 files changed

+18
-24
lines changed

2 files changed

+18
-24
lines changed

clang/lib/CIR/CodeGen/CIRGenModule.cpp

Lines changed: 17 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -508,57 +508,51 @@ void CIRGenModule::setDSOLocal(CIRGlobalValueInterface gv) const {
508508
}
509509

510510
const TargetCIRGenInfo &CIRGenModule::getTargetCIRGenInfo() {
511-
if (TheTargetCIRGenInfo)
512-
return *TheTargetCIRGenInfo;
511+
if (theTargetCIRGenInfo)
512+
return *theTargetCIRGenInfo;
513513

514-
// Helper to set the unique_ptr while still keeping the return value.
515-
auto SetCIRGenInfo = [&](TargetCIRGenInfo *P) -> const TargetCIRGenInfo & {
516-
this->TheTargetCIRGenInfo.reset(P);
517-
return *P;
518-
};
514+
const llvm::Triple &triple = getTarget().getTriple();
519515

520-
const llvm::Triple &Triple = getTarget().getTriple();
521-
522-
switch (Triple.getArch()) {
516+
switch (triple.getArch()) {
523517
default:
524518
assert(false && "Target not yet supported!");
525519

526520
case llvm::Triple::aarch64_be:
527521
case llvm::Triple::aarch64: {
528-
AArch64ABIKind Kind = AArch64ABIKind::AAPCS;
522+
AArch64ABIKind kind = AArch64ABIKind::AAPCS;
529523
assert(getTarget().getABI() == "aapcs" ||
530524
getTarget().getABI() == "darwinpcs" &&
531525
"Only Darwin supported for aarch64");
532-
Kind = AArch64ABIKind::DarwinPCS;
533-
return *(TheTargetCIRGenInfo =
534-
createAArch64TargetCIRGenInfo(genTypes, Kind));
526+
kind = AArch64ABIKind::DarwinPCS;
527+
return *(theTargetCIRGenInfo =
528+
createAArch64TargetCIRGenInfo(genTypes, kind));
535529
}
536530

537531
case llvm::Triple::x86_64: {
538-
StringRef ABI = getTarget().getABI();
539-
X86AVXABILevel AVXLevel = (ABI == "avx512" ? X86AVXABILevel::AVX512
540-
: ABI == "avx" ? X86AVXABILevel::AVX
532+
StringRef abi = getTarget().getABI();
533+
X86AVXABILevel avxLevel = (abi == "avx512" ? X86AVXABILevel::AVX512
534+
: abi == "avx" ? X86AVXABILevel::AVX
541535
: X86AVXABILevel::None);
542536

543-
switch (Triple.getOS()) {
537+
switch (triple.getOS()) {
544538
default:
545539
assert(false && "OSType NYI");
546540
case llvm::Triple::Linux:
547-
return *(TheTargetCIRGenInfo =
548-
createX86_64TargetCIRGenInfo(genTypes, AVXLevel));
541+
return *(theTargetCIRGenInfo =
542+
createX86_64TargetCIRGenInfo(genTypes, avxLevel));
549543
}
550544
}
551545

552546
case llvm::Triple::spirv64: {
553-
return *(TheTargetCIRGenInfo = createSPIRVTargetCIRGenInfo(genTypes));
547+
return *(theTargetCIRGenInfo = createSPIRVTargetCIRGenInfo(genTypes));
554548
}
555549

556550
case llvm::Triple::nvptx64: {
557-
return *(TheTargetCIRGenInfo = createNVPTXTargetCIRGenInfo(genTypes));
551+
return *(theTargetCIRGenInfo = createNVPTXTargetCIRGenInfo(genTypes));
558552
}
559553

560554
case llvm::Triple::amdgcn: {
561-
return *(TheTargetCIRGenInfo = createAMDGPUTargetCIRGenInfo(genTypes));
555+
return *(theTargetCIRGenInfo = createAMDGPUTargetCIRGenInfo(genTypes));
562556
}
563557
}
564558
}

clang/lib/CIR/CodeGen/CIRGenModule.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class CIRGenModule : public CIRGenTypeCache {
7474
const std::string &getModuleNameHash() const { return ModuleNameHash; }
7575

7676
private:
77-
mutable std::unique_ptr<TargetCIRGenInfo> TheTargetCIRGenInfo;
77+
mutable std::unique_ptr<TargetCIRGenInfo> theTargetCIRGenInfo;
7878

7979
/// The builder is a helper class to create IR inside a function. The
8080
/// builder is stateful, in particular it keeps an "insertion point": this

0 commit comments

Comments
 (0)