Allow an assembler backend to get ABI options. This is to match the changes to http://reviews.llvm.org/D16213.
Tested with "make check-clang"
Differential D16538
[cc1as] Add MCTargetOptions argument to createAsmBackend joelkevinjones on Jan 25 2016, 9:08 AM. Authored by
Details Allow an assembler backend to get ABI options. This is to match the changes to http://reviews.llvm.org/D16213. Tested with "make check-clang"
Diff Detail Event TimelineComment Actions Hi Joel, Since you're likely going to need to start filling in the Options there anyhow, can you lift the one from the TODO and start filling it in? -eric Comment Actions Did you mean remove the duplication from addPassesToEmitFile mentioned in the FIXME or something else? Comment Actions The FIXME comment doesn't make sense to me. I'm not sure what sanitizer arguments would be passed to the assembler. Comment Actions I think it refers to MCTargetOptions::SanitizeAddress but I don't know where we'd get that information from. Unfortunately, it seems that the original author (@ygorshenin) might not be on the list anymore (phabricator shows no activity since 2014) but we can ask Evgeniy.
|
@eugenis: Do you know what needs to be done for this FIXME?