[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