[PATCH] D62997: [LV] Share the LV illegality reporting with LoopVectorize. NFC.
Hideki Saito via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 17 10:58:47 PDT 2019
hsaito added a comment.
In D62997#1589461 <https://reviews.llvm.org/D62997#1589461>, @psamolysov wrote:
> The test `no_switch.ll` has been updated: the LV doesn't report remarks without `-pass-remarks-missed='loop-vectorize' -pass-remarks-analysis='loop-vectorize'`, so the test from the CHECK section is removed. I have no idea why, but remarks with lambdas through the code emitted via `OptimizationRemarkMissed`, not `OptimizationRemarkAnalysis` (if I right understand, 'Missed' means the pass has not been applied by any reasons during optimization).
Thanks, @psamolysov. No more LIT test failures? I don't understand the issue with no_switch.ll either, but trying to understand that shouldn't block this patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62997/new/
https://reviews.llvm.org/D62997
More information about the llvm-commits
mailing list