[PATCH] D34096: [Sema][C++1z] Ensure structured binding's bindings in dependent foreach have non-null type

[Benjamin Buch via Phabricator via cfe-commits]

bebuch reopened this revision.
bebuch added a comment.
This revision is now accepted and ready to land.

I believe this patch is incomplete, I get a very odd warning:

  struct A{
      int x;
  };
  
  template < typename >
  struct B{
      A data_[1];
  
      void f(){
          for(auto [x]: data_){
              (void)x;
          }
      }
  };
  
  int main(){
        B< int > v;
        v.f();
  }



  $ clang++ -std=c++1z -Wall clang_fail.cpp                                     
                                                                                
                                                           
  clang_fail.cpp:10:18: warning: unused variable '' [-Wunused-variable]         
                                                                                
                                                                                
                        
          for(auto [x]: data_){                                                 
                                                                                
                                                                                
                        
                   ^                                                            
                                                                                
                                                                                
                        
  clang_fail.cpp:18:4: note: in instantiation of member function 'B<int>::f' 
requested here                                                                  
                                                                                
                           
          v.f();                                                                
                                                                                
                                                                                
                        
            ^                                                                   
                                                                                
                                                                                
                        
  1 warning generated.

unused variable ''? An unnamed variable, that's odd ;-)

Nevertheless, thanks for what is already done! Great that I can use structured 
bindings without fear to crash now! :-)


Repository:
  rL LLVM

https://reviews.llvm.org/D34096



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