[PATCH] D83655: [AsmPrinter] Split up .gcc_except_table

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 12:53:29 PST 2020


dblaikie added a comment.

(FWIW, I'd personally be in favor of -funique-section-names applying here - the specific textual description of the flag in documentation I don't think should necessarily be held as definitive, but possibly descriptive (perhaps it describes correctly the behavior as it exists/was implemented, and could be updated to include this new behavior) - seems like if someone's passing that flag it's probably because they don't want or may not have toolchain support for non-unique section names. It'd be pretty quirky/odd situation where they didn't want unique section names only for some sections and not for others.

But I don't feel super strongly about it if folks have other ideas)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83655



More information about the llvm-commits mailing list