vitalybuka accepted this revision.
vitalybuka added a comment.
This revision is now accepted and ready to land.
my editor recommends me to keep git titles <50 chars
Maybe something shorter with exactly the same meaning
"[NFC] Optimize noundef analysis condition"
================
Comment at: clang/lib/CodeGen/CGCall.cpp:2380-2381
// Decide whether the argument we're handling could be partially undef
- bool ArgNoUndef = DetermineNoUndef(ParamType, getTypes(), DL, AI);
- if (CodeGenOpts.EnableNoundefAttrs && ArgNoUndef)
- Attrs.addAttribute(llvm::Attribute::NoUndef);
+ if (CodeGenOpts.EnableNoundefAttrs) {
+ if (DetermineNoUndef(ParamType, getTypes(), DL, AI))
+ Attrs.addAttribute(llvm::Attribute::NoUndef);
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117078/new/
https://reviews.llvm.org/D117078
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
