[PATCH] D156537: [CodeGen] Keep track of eagerly emitted globals

Vassil Vassilev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 15 12:23:15 PDT 2023


v.g.vassilev added a comment.

In D156537#4587874 <https://reviews.llvm.org/D156537#4587874>, @aaron.ballman wrote:

> In D156537#4587832 <https://reviews.llvm.org/D156537#4587832>, @v.g.vassilev wrote:
>
>> @Hahnfeld, let's move forward and rely on a post-commit review here if necessary.
>
> Because one of the codegen code owners has been active on the thread with concerns, I think it's better to wait a bit longer for an explicit sign-off that those concerns are addressed (this code impacts more than clang-repl).

D157379 <https://reviews.llvm.org/D157379> seems to resolve the concern but indeed we can wait a little longer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156537



More information about the cfe-commits mailing list