Michael137 wrote:

> LGTM You might want to use `%enum_select` to improve that further

Ah nice didn't know about `enum_select`

The enum is currently unnamed and local to `SemaExpr` so we'd have to move it 
out some header, name it, etc. So to keep the diff small I'll leave that for 
possible follow-up

https://github.com/llvm/llvm-project/pull/179895
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to