Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/ExecutionEngine/SparseUtils.cpp | ||
---|---|---|
665 | while you are here, how do you feel about using getNext here (the base name of the method, not the macro)? |
Paths
| Differential D112771
[mlir][sparse] Renaming CPP macros for clarity ClosedPublic Authored by wrengr on Oct 28 2021, 5:21 PM.
Details
Diff Detail
Event TimelineHerald added subscribers: wenzhicui, Chia-hungDuan, dcaballe and 17 others. · View Herald TranscriptOct 28 2021, 5:21 PM This revision is now accepted and ready to land.Oct 28 2021, 8:11 PM Closed by commit rG30a64c9aa558: [mlir][sparse] Renaming CPP macros for clarity (authored by wrengr). · Explain WhyOct 29 2021, 1:08 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 383478 mlir/lib/ExecutionEngine/SparseUtils.cpp
|
while you are here, how do you feel about using getNext here (the base name of the method, not the macro)?