[PATCH] D37386: [AMDGPU] Implement infrastructure to set options in AMDGPUToolChain

[Andrey Kasaurov via Phabricator via cfe-commits]

kasaurov added inline comments.


================
Comment at: lib/Driver/ToolChains/AMDGPU.h:60
   bool IsIntegratedAssemblerDefault() const override { return true; }
+  llvm::opt::DerivedArgList *
+  TranslateArgs(const llvm::opt::DerivedArgList &Args, StringRef BoundArch,
----------------
emankov wrote:
> Return arg on the same line.
That's how clang-format sees it: 
-DerivedArgList *AMDGPUToolChain::TranslateArgs(const DerivedArgList &Args, StringRef BoundArch, Action::OffloadKind DeviceOffloadKind) const {
+DerivedArgList *
+AMDGPUToolChain::TranslateArgs(const DerivedArgList &Args, StringRef BoundArch,
+                               Action::OffloadKind DeviceOffloadKind) const {



https://reviews.llvm.org/D37386