Typz added a comment.

Regarding `?:`, this is definitely not considered, and should be relatively 
easy to handle.

Regarding case `D`, this looks like a bug... The break in the first operand is 
kind of unexpected (as in "not the usual case"), and I had many such corner 
cases while making and testing the change. I thought I finally had a proper 
solution, seems there was still something, though I am not sure it is related 
to `?:` or could happen in other cases (on second thought, it may indeed be 
related, as there is no operand to align with...)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D50078/new/

https://reviews.llvm.org/D50078



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to