[PATCH] D108138: [WIP] Remove switch statements before vectorization
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 15 09:28:44 PDT 2021
lebedev.ri added a comment.
Hi. I'm personally still not very okay with the approach as it currently is.
Do you need to run LoopRotate after lowering switches? Anything else?
But then you don't actually know that after spending all this compile time,
the vectorization will actually happen, and you won't just now need to undo all this,
correct? This seems conceptually wrong to me.
Will LV never have to learn to deal with switches properly?
I would assume it will, in which case what is the urgency of this temporary approach?
If you really don't want to fix this properly, i'm looking forward to an RFC on llvm-dev.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108138/new/
https://reviews.llvm.org/D108138
More information about the cfe-commits
mailing list