[PATCH] D70076: [Polly][CodeGen]Fix getArrayAccessFor crashes as in bug 32534 with -polly-vectorizer=polly

bin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 04:55:57 PST 2019


bin.narwal added a comment.

In D70076#1741683 <https://reviews.llvm.org/D70076#1741683>, @Meinersbur wrote:

> Thanks for looking into the bug. The suggestion change is obvious (IMHO).
>
> However, the test case seems to be the unmodified reduced test case from bugzilla. Generally, committed test cases should be
>
> 1. Minimal (No unnecessary comments, metadata, etc.)
> 2. Have at least one positive CHECK line.
>
>   But the change is obvious enough, I'd commit it without test-case.


Okay, thanks.  Please help commit it.  I will try to minimize the test case in the future.

Thanks,


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70076





More information about the llvm-commits mailing list