Thank you. Please use

Changed variable case

Removed superfluous newline.

Thanks, that sounds more reasonable. I closed the other differential and updated this one with a squashed commit.

I added a testcase and ran arcanist again, which created a child differential D139460. I hope that's right.

Dear LLVM developers -- this is my first contribution to the LLVM project, I hope I got it right.

Disclaimer: I am not an LLVM developer. Just looking at the patch, I wonder if it could be much less intrusive if LValue CodeGenFunction::EmitLValue(const Expr *E) is split into two methods -- a private one, and a public-facing one that applies your ClearTBAA method.

FYI: I gave it a try, and it looks like the union aliasing-related issue mentioned in Bug 32056 is not resolved by this patch.