In order to pass the ABI info, MCTargetOptions parameter is added to createAsmBackend methods. This patch introduces necessary changes to clang driver to adopt this change.
Not right now, at least not a simple one. The FIXME comment that was removed said:"init MCTargetOptions from sanitizer flags here." So we should probably need to map clang options to MCTargetOptions.
Could we do that in a separate patch? I would prefer to submit this one without initialization with old fixme comment and commit it in order to get the abi working. In the next patch we can initialize MCtargetOptions from AssemblerInvocation(adding the fields we need in the structure).
From: Eric Christopher [firstname.lastname@example.org]
Sent: Friday, February 20, 2015 6:54 PM
To: reviews+D7747email@example.com; Vladimir Medic; Daniel Sanders
Subject: Re: [PATCH] [clang][Driver][MCTargetOptions]Change clang driver to adopt new definition of createAsmBackend
MCTargetOptions instance is not initiated anymore using InitMCTargetOptionsFromFlags but newly defined method which uses AssemblerInvocation structure to populate fields from clang driver options.