dexonsmith added inline comments.
================
Comment at: clang/lib/Basic/SourceManager.cpp:167
Buffer.setInt(Buffer.getInt() | InvalidFlag);
+ return None;
----------------
dexonsmith wrote:
> JDevlieghere wrote:
> > Orthogonal to this patch but an idea for a follow-up: this seems lightly
> > error prone and repetitive. We could put it in an `llvm::scope_exit` and
> > then release it before the final return on line `200`.
> Good call; I refactored this a bit in https://reviews.llvm.org/D89431; after
> that lands I'll do what you suggest, or possibly start with `IsBufferInvalid
> = true` and then set it back to `false` at the end of the function (once we
> know it's good).
Posted the follow-up at https://reviews.llvm.org/D89722.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89430/new/
https://reviews.llvm.org/D89430
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
