[PATCH] D78495: [nfc] Accept addrspacecast allocas in InitTempAlloca

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 20 08:37:57 PDT 2020


JonChesterfield added a comment.

In D78495#1992404 <https://reviews.llvm.org/D78495#1992404>, @arsenm wrote:

> Needs test?


I'm not sure how to write said test. How do we normally hit asserts from the clang test suite?

This fires a lot in the openmp on amdgcn downstream branch, but I'm happy carrying this as a local patch until the rest of the clang change can be put up for review if preferred.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78495





More information about the cfe-commits mailing list