llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) <details> <summary>Changes</summary> Since both alignment and Alignment are of the same type, this patch renames alignment to Alignment while removing the cast statement. --- Full diff: https://github.com/llvm/llvm-project/pull/146622.diff 1 Files Affected: - (modified) clang/lib/Sema/SemaAttr.cpp (+1-2) ``````````diff diff --git a/clang/lib/Sema/SemaAttr.cpp b/clang/lib/Sema/SemaAttr.cpp index c1675a6c67f14..3eed6ad7fe6b3 100644 --- a/clang/lib/Sema/SemaAttr.cpp +++ b/clang/lib/Sema/SemaAttr.cpp @@ -436,7 +436,7 @@ void Sema::ActOnPragmaClangSection(SourceLocation PragmaLoc, } void Sema::ActOnPragmaPack(SourceLocation PragmaLoc, PragmaMsStackAction Action, - StringRef SlotLabel, Expr *alignment) { + StringRef SlotLabel, Expr *Alignment) { bool IsXLPragma = getLangOpts().XLPragmaPack; // XL pragma pack does not support identifier syntax. if (IsXLPragma && !SlotLabel.empty()) { @@ -445,7 +445,6 @@ void Sema::ActOnPragmaPack(SourceLocation PragmaLoc, PragmaMsStackAction Action, } const AlignPackInfo CurVal = AlignPackStack.CurrentValue; - Expr *Alignment = static_cast<Expr *>(alignment); // If specified then alignment must be a "small" power of two. unsigned AlignmentVal = 0; `````````` </details> https://github.com/llvm/llvm-project/pull/146622 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
