lebedev.ri added a comment.
Thanks, i like it.
================
Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:2058-2073
+/*
void CodeGenFunction::EmitAlignmentAssumption(llvm::Value *PtrValue,
QualType Ty, SourceLocation Loc,
SourceLocation AssumptionLoc,
unsigned Alignment,
llvm::Value *OffsetValue) {
llvm::Value *TheCheck;
----------------
Just remove.
================
Comment at: clang/lib/CodeGen/CodeGenFunction.h:2831-2836
+/*
void EmitAlignmentAssumption(llvm::Value *PtrValue, QualType Ty,
SourceLocation Loc, SourceLocation
AssumptionLoc,
unsigned Alignment,
llvm::Value *OffsetValue = nullptr);
+*/
----------------
Just remove.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:6066
<< Arg->getSourceRange();
+ if (Result > (1 << 29ULL))
+ Diag(TheCall->getBeginLoc(), diag::warn_assume_align_too_big)
----------------
`(1ULL << 29ULL)`.
Is there some define for this somwhere, don't like magic numbers.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68824/new/
https://reviews.llvm.org/D68824
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
