[PATCH] D111293: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

[John McCall via Phabricator via cfe-commits]

rjmccall added a comment.

We should take the opportunity to add a test here for the ObjC case.  It should 
be testable with something like this, where you should see the store inside of 
the loop instead of once in the prologue:

  // RUN: %clang_cc1 -triple x86_64-unknown-freebsd -S -emit-llvm 
-fobjc-runtime=gnustep-2.0 -o - %s | FileCheck %s
  
  typedef struct {
    int x[12];
  } Big;
  
  @protocol P
  - (Big) foo;
  @end
  
  void test(id<P> p) {
    for (;;) {
      Big big = [p foo];
    }
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111293

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits