[PATCH] D104078: [Remarks] Make memsize remarks report as an analysis, not a missed opportunity.
Francis Visoiu Mistrih via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 21 17:18:41 PDT 2021
thegameg accepted this revision.
thegameg added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/include/llvm/Transforms/Utils/MemoryOpRemark.h:59
+
+ enum RemarkStyle { RS_Missed, RS_Analysis };
+ virtual RemarkStyle remarkStyle() const { return RS_Analysis; }
----------------
Can we reuse `enum DiagnosticKind` from DiagnosticInfo.h?
```
DK_OptimizationRemarkMissed,
DK_OptimizationRemarkAnalysis,
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104078/new/
https://reviews.llvm.org/D104078
More information about the llvm-commits
mailing list