[PATCH] D38325: [AMDGPU] Set fast-math flags on functions given the options

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 29 15:24:31 PDT 2017


dfukalov added a comment.

otherwise LGTM



================
Comment at: lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:375
     PassManagerBuilder::EP_EarlyAsPossible,
-    [AMDGPUAA, LibCallSimplify](const PassManagerBuilder &,
-                                legacy::PassManagerBase &PM) {
+    [AMDGPUAA, LibCallSimplify, this](const PassManagerBuilder &,
+                                      legacy::PassManagerBase &PM) {
----------------
It possible would be better to create temporary ref for `Options` member and capture it instead of whole `this` pointer


https://reviews.llvm.org/D38325





More information about the llvm-commits mailing list