jyknight added a comment.

I wonder if we actually need to define a clang frontend flag for this; I 
suspect nobody will ever want to specify it, since the only non-canonical 
personality clang will ever generate that changes behavior is the pure-C 
destructor-only personality, `__gnu_personality_v0`. Otherwise, it could only 
arise from assembly or IR.

For a test-case, can just use `-mllvm -emit-compact-unwind-non-canonical`.



================
Comment at: llvm/lib/MC/MCAsmBackend.cpp:119
+
+bool MCAsmBackend::isDarwinCanonicalPersonality(const MCSymbol *Sym) const {
+  if (Sym && Sym->isMachO()) {
----------------
This could use a comment as to why these two are "canonical" and why 
"__gcc_personality_v0" is excluded despite being similar.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144999

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

Reply via email to