Followup from: https://reviews.llvm.org/D43399
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM
wasm/Symbols.h | ||
---|---|---|
124–125 ↗ | (On Diff #134735) | If you do not = nullptr, please remove it as well. |
wasm/Symbols.h | ||
---|---|---|
124–125 ↗ | (On Diff #134735) | Ha.. already did that but hadn't uploaded it yet. Great minds .. |
Comment Actions
Thanks for splitting this one out two, reduces the diff on my patch nicely! With this time delay, you can get onto these changes 24h ahead - which I don't mind, but I will get round to it for work that I've generated if you leave it a day (or two...)