[PATCH] D110061: [SelectionDAG] Remove PromoteIntOp_EXTRACT_SUBVECTOR.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 08:51:47 PDT 2021


craig.topper added a comment.

In D110061#3011500 <https://reviews.llvm.org/D110061#3011500>, @sdesmalen wrote:

> In D110061#3010295 <https://reviews.llvm.org/D110061#3010295>, @craig.topper wrote:
>
>> I'm not sure we can write a test for RISCV either. This can only happen if the element type needs to be promoted on the larger vector type, but not the narrower vector type used for the result. I don't even know to exercise this code with fixed vectors. The llvm-cov bot says its untested http://lab.llvm.org:8080/coverage/coverage-reports/coverage/Users/buildslave/jenkins/workspace/coverage/llvm-project/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp.html#L5001.  Looks like it was added to support some case with mask vectors on X86, but that case no longer happens. Maybe we can just delete it?
>
> Thanks, I wasn't aware of a code-coverage buildbot that actually showed this info, that's very useful. Is there a more friendly interface to get such a report? (Other than manually constructing the URL with a path to the file)

There's an "llvm-cov" link in the left hand column on llvm.org that should take you to a list of all files.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110061/new/

https://reviews.llvm.org/D110061



More information about the llvm-commits mailing list