[PATCH] D142606: Lazyly initialize uncommon toolchain detector

[Timm Bäder via Phabricator via cfe-commits]

tbaeder added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/LazyDetector.h:36
+    }
+    return &Detector.value();
+  }
----------------
Is the `.value()` here permitted? AFAIK the convention is to always use `operator*`.

And I think @aaron.ballman would tell you to drop the `{}` around that single-statement `if` :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142606/new/

https://reviews.llvm.org/D142606