jmciver added inline comments.
================ Comment at: clang/lib/CodeGen/CGExpr.cpp:676 +namespace { +void applyNoundefToLoadInst(bool enable, const clang::QualType &Ty, + llvm::LoadInst *Load) { ---------------- tschuett wrote: > Nit: You meant static. Thank you for bringing that to my attention. I reviewed the [[ https://llvm.org/docs/CodingStandards.html#anonymous-namespaces | LLVM Coding Standards ]] guidance on the topic. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134410/new/ https://reviews.llvm.org/D134410 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits