arsenm added inline comments.
================ Comment at: llvm/include/llvm/Transforms/Utils/InferCallsiteAttrs.h:1 +#ifndef LLVM_TRANSFORMS_UTILS_INFERCALLSITEATTRS_H +#define LLVM_TRANSFORMS_UTILS_INFERCALLSITEATTRS_H ---------------- Missing license header ================ Comment at: llvm/include/llvm/Transforms/Utils/InferCallsiteAttrs.h:10-12 + static constexpr uint8_t kMaybe = 0; + static constexpr uint8_t kYes = 1; + static constexpr uint8_t kNo = 2; ---------------- Why not a regular enum? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152226/new/ https://reviews.llvm.org/D152226 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
