[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

2021-10-11 Thread Mahesha S via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGdb9c2d775130: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca() (authored by hsmhsm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https:

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

2021-10-08 Thread Mahesha S via Phabricator via cfe-commits
hsmhsm updated this revision to Diff 378413. hsmhsm added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https://reviews.llvm.org/D111324 Files: clang/lib/CodeGen/CGExpr.cpp clang/lib/CodeGen/CodeGenFunction.h I

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

2021-10-08 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Thanks. This LGTM when all the patches it depends on are checked in. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https://revi

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

2021-10-07 Thread Mahesha S via Phabricator via cfe-commits
hsmhsm updated this revision to Diff 378094. hsmhsm added a comment. Move OpenMP GPU offloading codegen related changes from here to https://reviews.llvm.org/D111324. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https://reviews.llvm.o

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

2021-10-07 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. Can you just do one patch for all the OpenMP changes, and then put the final removal in a separate patch? Thanks for taking this on. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https://reviews.llvm.org/D111