ZarkoCA added inline comments.

================
Comment at: clang/lib/CodeGen/CGDeclCXX.cpp:691
   AddGlobalDtor(Fn);
+  CXXGlobalDtors.clear();
 }
----------------
I may be missing something but why do we need this now, as opposed to not 
needing it before? Why didn't we need to clear the CXXGlobalDtors after 
emitting the function function before?


================
Comment at: clang/lib/CodeGen/ItaniumCXXABI.cpp:4442
+  if (D.getTLSKind() != VarDecl::TLS_None)
+    llvm::report_fatal_error("Thread local storage unimplemented on AIX yet.");
+
----------------
Style guide now says no capitalization and no punctuation at the end for 
`report_fatal_error` messages.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74166



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

Reply via email to