[PATCH] D93375: [clang][driver] Add -ansi option to CompileOnly group

2021-01-12 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. Thanks for the commits, Aaron. If you don't mind I have a short question: both commits don't seem to list the test cases, have they not been committed at all or are they filtered out in github and/or phabricator? Did I mess something up? CHANGES SINCE LAST ACTION htt

[PATCH] D84846: [MC] Add support for generating missing GNU build notes

2021-01-15 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder updated this revision to Diff 316944. Herald added a subscriber: martong. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84846/new/ https://reviews.llvm.org/D84846 Files: clang/include/clang/AST/Expr.h clang/include/clang/AST/RecursiveASTVisitor.h clang/include/clang/Basic/S

[PATCH] D84846: [MC] Add support for generating missing GNU build notes

2021-01-15 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder updated this revision to Diff 316945. tbaeder added a comment. Sorry about the last change, that was the wrong patch. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84846/new/ https://reviews.llvm.org/D84846 Files: clang/include/clang/Basic/CodeGenOptions.def clang/include/c

[PATCH] D94873: [clang] [driver] Remove obsolete unistd.h include

2021-01-16 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder created this revision. tbaeder added reviewers: rsmith, aaron.ballman, dlj. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This was needed for a call to `getuid()`, but that call is gone these days, so remove the include a

[PATCH] D94873: [clang] [driver] Remove obsolete unistd.h include

2021-01-19 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. Pushed as b86e7ae66cb988dda33445c29fa64f93e7ca9c3c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94873/new/ https://reviews.llvm.org/D94873 ___

[PATCH] D108881: [clang][driver] Honor the last -flto= flag even if an earlier -flto is present

2021-08-29 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added a comment. That code has changed quite a bit since I've worked on it. The only problem I could see is that passing `-flto=thin -flto` and choosing thin LTO kinda makes sense if you interpret `-flto` as just "use LTO". The `-flto=thin` is more specific, so it could make sense to pi

<    17   18   19   20   21   22