jfb added a comment. In D58218#1398251 <https://reviews.llvm.org/D58218#1398251>, @rjmccall wrote:
> In D58218#1398124 <https://reviews.llvm.org/D58218#1398124>, @jfb wrote: > > > In D58218#1398096 <https://reviews.llvm.org/D58218#1398096>, @rjmccall > > wrote: > > > > > Well, you can always make a variable with a more directly-applicable name > > > than `capturedByInit` and update it as appropriate, like > > > `locIsByrefHeader`. > > > > > > Sounds good. I made it `const` too, to avoid inadvertent modification. > > > This is actually the reverse of the sense I would expect it to have based on > the name. If you want these semantics, maybe `locIsObject`? You're right, I've fixed the polarity. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58218/new/ https://reviews.llvm.org/D58218 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
