This is for compatibility with MSVC, which also marks them as const. This is a break-off of D54667 to try to get rid of some of the boilerplate from the patch.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Right, just for the type of this variables. I... thought we already fixed this? Or maybe I just filed the bug and never actually submitted code to fix it? Whatever, ship the fix and I'll worry about my own deja-vu some other time. lgtm
Comment Actions
Hey, I'm not crazy: https://bugs.llvm.org/show_bug.cgi?id=36526 I filed the bug but never followed up. Mark it closed for extra points.