[cfe-commits] r146413 - in /cfe/trunk: include/clang/Basic/BuiltinsHexagon.def include/clang/Basic/LangOptions.def include/clang/Basic/TargetBuiltins.h include/clang/Driver/CC1Options.td include/clang/Driver/Options.td lib/Basic/Targets.cpp lib/C

Eli Friedman eli.friedman at gmail.com
Mon Dec 19 18:43:01 PST 2011


On Mon, Dec 19, 2011 at 1:02 PM,  <jverma at codeaurora.org> wrote:
> Hi All,
>
> I have made the changes to remove assertions in the Hexagon backend
> specific clang driver. Instead of asserting on invalid arch name, it has
> been modified to use the default value.
>
> Please find attached the patch file for the fix.

Silently ignoring unexpected options is confusing for users; a
diagnostic would be better.

-Eli



More information about the cfe-commits mailing list