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
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.