[PATCH] D62997: [LV] Share the LV illegality reporting with LoopVectorize. NFC.

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 11:43:52 PDT 2019


hsaito added a comment.

Aha, that was a "hacky" way to get "loop contains a switch statement" along with the warning. I see. I suppose we can't blindly use LV_NAME, then.
I then suppose some tests (like pr38800.ll) didn't even need -pass-remarks-missed flag (which is incorrectly used, I think).

We better add some comments in vectorizeAnalysisPassName().


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