[PATCH] [clang][Driver][MCTargetOptions]Change clang driver to adopt new definition of createAsmBackend

Vladimir Medic Vladimir.Medic at imgtec.com
Fri Feb 20 06:09:54 PST 2015


================
Comment at: tools/driver/cc1as_main.cpp:358
@@ -357,2 +357,3 @@
       TheTarget->createMCSubtargetInfo(Opts.Triple, Opts.CPU, FS));
+  MCTargetOptions Options = InitMCTargetOptionsFromFlags();
 
----------------
echristo wrote:
> I'd prefer not to do this, clang already has its own option handling. Ideas on how to avoid this?
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.

http://reviews.llvm.org/D7747

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list