DavidSpickett added a comment.
Looks ok but needs tests. See clang/test/Driver for examples of checking -###
output.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:1525
+void AddAAPCSVolatileBitfieldArgs(const ArgList &Args, ArgStringList &CmdArgs)
{
+ if (!Args.hasFlag(options::OPT_faapcs_bitfield_width,
----------------
Name this AddARMAAPCSVolatileBitfieldArgs to match the other helpers.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96784/new/
https://reviews.llvm.org/D96784
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
