[clang] [Clang][NFC] Clarify some SourceManager related code (PR #153527)

[Aaron Ballman via cfe-commits]

================
@@ -1095,6 +1095,8 @@ prepareAndFilterRanges(const SmallVectorImpl<CharSourceRange> &Ranges,
 
     unsigned StartColumn = SM.getExpansionColumnNumber(Begin);
     unsigned EndColumn = SM.getExpansionColumnNumber(End);
+    assert(StartColumn && "StartColumn has a value of 0");
----------------
AaronBallman wrote:

I think the request for something like: `assert(StartColumn && "StartColumn must be valid, 0 is invalid");`

https://github.com/llvm/llvm-project/pull/153527