rjmccall added inline comments.

================
Comment at: clang/lib/CodeGen/ItaniumCXXABI.cpp:2370
+  if (CGM.getCodeGenOpts().CXAAtExit || D.getTLSKind())
     return emitGlobalDtorWithCXAAtExit(CGF, dtor, addr, D.getTLSKind());
 
----------------
It's fine to match the GCC behavior, and it's even somewhat explicable.  Could 
you update the comment to explain, first of all, that this path handles both 
`__cxa_atexit` and `__cxa_thread_atexit`, and secondly that the option is only 
meant to control the former?


Repository:
  rC Clang

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

https://reviews.llvm.org/D63283



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

Reply via email to