[PATCH] D122258: [MC] Omit DWARF unwind info if compact unwind is present where eligible
Shoaib Meenai via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 5 11:41:40 PDT 2022
smeenai added inline comments.
================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:456
Options.MCOptions.SplitDwarfFile = CodeGenOpts.SplitDwarfFile;
+ Options.MCOptions.EmitDwarfUnwind = CodeGenOpts.getEmitDwarfUnwind();
Options.MCOptions.MCRelaxAll = CodeGenOpts.RelaxAll;
----------------
int3 wrote:
> this code doesn't execute if clang is passed an assembly file instead of a .c file, so this option doesn't have the desired effect on assembly inputs. I'm not sure what's the right way to tackle this, or if this behavior inconsistency is acceptable
>
>
It seems unfortunate to have that inconsistency. From what I can tell, clang/tools/driver/cc1as_main.cpp looks like it might be the rough equivalent of this for the integrated assembler?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122258/new/
https://reviews.llvm.org/D122258
More information about the cfe-commits
mailing list